Segmentation fault bei Linux

[Erik (Gast)]

Hallo allerseits,

warum liefert der folgende code einen "segmentation fault error", er 
wird unter linux Suse 10.1 compiliert und ausgeführt.

#include <mysql/mysql.h>
#include <stdio.h>
#include <string.h>

int main()
{
   MYSQL mysql;
   MYSQL_RES *res;
   MYSQL_ROW row;

   char query[80];
   mysql_init(&mysql);
   mysql_real_connect(&mysql,"localhost","yyy","xxx","test",0,NULL,0);
   sprintf(query,"SELECT * FROM test");
   mysql_real_query(&mysql,query,(unsigned int)strlen(query));
   res = mysql_use_result(&mysql);
   while(row = mysql_fetch_row(res))
    printf("%s %sn",row[0],row[1]);
   mysql_free_result(res);
   return 0;
}


Was muss geändert werden ?
Danke im Voraus !

[Ulrich (Gast)]

als erstes würde ich mal versuchen rauszufinden an welche rStelle der 
Fehlerpassiert. Schonmal den gdb angeworfen?

[Μαtthias W. (matthias)]

Hi

oder erstmal anfangen auch nur ein Mindestmaß an Fehlerbehandlung 
einzubauen. Wer sagt denn das mysql_init(&mysql) ff. überhaupt 
erfolgreich sind.

Matthias

[Andreas S. (andreas) (Admin)]

gdb deinprogramm
(gdb) run
[Fehlermeldung]
(gdb) backtrace

[Niquo (Gast)]

Also
   MYSQL mysql;
   mysql_init(&mysql);
sieht schon mal ganz böse aus, weil mysql_init doch den Speicher 
allozieren will. Demnach würde ich mal das hier versuchen:
   MYSQL *mysql = (MYSQL*) 0;
   mysql_init(mysql);

[A.K. (Gast)]

Unabhängig vom konkreten Problem zum Stil: Stichwort "Defensive 
Programmierung". Geh davon aus, dass Funktionen, die Fehler 
zurückliefern können, es auch ab und zu tun werden.

[Erik (Gast)]

Danke erstmal, seitens der DB müsste es klappen, der obige code ist 
nicht der einzige den ich probiert habe.

MYSQL *mysql = (MYSQL*) 0; ändert leider nichts.


etwas länger und "sicherer" und ebenfalls segfault:

#include <mysql/mysql.h>
#include <stdio.h>
#include <string.h>

int main(){
   MYSQL *mysql;
   MYSQL_RES *res;
   MYSQL_ROW row;
   char *query;
   int t,r;

   mysql_init(mysql);
   if (!mysql_real_connect(mysql,"localhost","apc0", 
"xxx","test",0,NULL,0))
   {
       printf( "Error connectin ot database: %s\n",mysql_error(mysql));
   }
   else printf("Connected...\n");

   query="select * from test";

   t=mysql_real_query(mysql,query,(unsigned int) strlen(query));
   if (t)
   {
      printf("Error making query: %s\n",
              mysql_error(mysql));
   }
   else printf("Query made...\n");
   res=mysql_use_result(mysql);
   for(r=0;r<=mysql_field_count(mysql);r++){
           row=mysql_fetch_row(res);
           if(row<0) break;
           for(t=0;t<mysql_num_fields(res);t++){
                   printf("%s ",row[t]);
           }
           printf("\n");
   }
   mysql_close(mysql);
return 0;
}


gdb liefert hierzu

Program received signal SIGSEGV, Segmentation fault.
0xb7d200e7 in memset () from /lib/libc.so.6
(gdb) backtrace
#0  0xb7d200e7 in memset () from /lib/libc.so.6
#1  0xb7e0e70a in mysql_init () from /usr/lib/libmysqlclient.so.15
#2  0x08048742 in main ()


Hmm, und was sagt uns das ?

[Andreas S. (andreas) (Admin)]

Dass memset(), welches von mysql_init() aufgerufen wurde, welches 
wiederum von main() aufgerufen wurde, auf einen ungültigen 
Speicherbereich zugreift. Der Grund ist, dass du für mysql keinen 
Speicher reserviert hast, mysql_init aber davon ausgeht dass es sich um 
einen gültigen Zeiger handelt. Die Lösung hat dir Niquo schon verraten, 
und sie steht auch in der Dokumentation zu mysql_init(): "If mysql is a 
NULL pointer, the function allocates, initializes, and returns a new 
object." Du musst den Zeiger also mit 0 (bzw. dem Makro NULL, zur 
besseren optischen Unterscheidung zwischen Integern und Zeigern) 
initialisieren:

MYSQL *mysql = NULL;

[A.K. (Gast)]

Da war die erste Version besser. Die zweite verwendet einen 
undefinierter Pointer (=> crash in memset). Entweder Platz übergeben 
(1.Version) oder NULL (Niquo).

[Erik (Gast)]

Danke für die Hilfe, sowie ich

MYSQL *mysql = 0;

verwende erhalte ich folgendes:

Program received signal SIGSEGV, Segmentation fault.
0xb7dd5c54 in mysql_real_connect () from /usr/lib/libmysqlclient.so.15
(gdb) backtrace
#0  0xb7dd5c54 in mysql_real_connect () from 
/usr/lib/libmysqlclient.so.15
#1  0x0804878c in main ()
(gdb)


Was ich nicht verstehe ist, dass dies alles Beispiel-codes von MySQL 
Seiten sind, die können doch nicht völlig "daneben" sein ?!?

[Erik (Gast)]

Habe A.Ks post erst jetzt gelesen - und bin nun endgültig verwirrt, was 
passt denn nun ?

[FBI (Gast)]

Lies Dir die Docs von MySQL durch!

Es gehen beide Varianten von mysql_init.
Aber nur wenn man sie auch richtig benutzt!!!
Also enweder:

MYSQL mysql;
mysql_init(&mysql);

oder:

MYSQL *pmysql;
pmysql = mysql_init(NULL);
if(!pmysql) {
  // ERROR
}

bzw.

MYSQL *pmysql = NULL;
pmysql = mysql_init(pmysql);
if(!pmysql) {
  // ERROR
}

CU Frank

[Erik (Gast)]

Ok, vielen Dank, hab´s endlich zum Laufen gebracht, war nach dem MYSQL 
°pmysql=NULL Problem noch ein anderer segfault Fehler im obigen code.

Vielen Dank an alle !

[FBI (Gast)]

>anderer segfault ...

Ja vermutlich beim printf :-)
Besser wär wohl gewesen:

printf("%s ", row[t] ? row[t] : "<NULL>");

Steht aber auch so direkt als Beispiel in der Docu zu mysql_fetch_row.

CU