C / Sqlite3 : SQL error: (null), Segmentation fault (core dumped)

Version imprimable

Bonjour,

Je me suis lancé dans la création d'un programme en C avec sqlite3 pour la base de donnée.

J'arrive à ouvrir la base de donnée à partir du C mais j'ai un problème lorsque je j'essaye d'insérer des donnée dans une table.

Terminal :

Code:

1
2
3
4
5
6
7
8
9
 
Opened database successfully
SQL error: (null)
Segmentation fault (core dumped)
 
 
------------------
(program exited with code: 139)
Press return to continue

Après quelques test j'en ai conclu que l'erreur vient de cette partie du code (quand je la met en commentaire je n'ai pas d'erreur) :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
 
 
static int callback(void *NotUsed, int argc, char **argv, char **azColName)
{
	int i;
	for(i=0; i<argc; i++){
	printf("%s = %s\n", azColName[i], argv[i] ? argv[i] : "NULL");
	}
	printf("\n");
	return 0;
}
 
////////////  PARTIE QUI POSE PROBLEME SELON MOI ///////////////
void SQLnewExercise(int Id,char *Name)
{
	sqlite3 *db;
	char *zErrMsg = 0;
	int rc;
	char *sql;
 
	/* Create SQL statement */
	sql="INSERT INTO Exercises (Id,Name)"\
		"VALUES (3, '%s');",Name;
 
	/* Execute SQL statement */
	rc = sqlite3_exec(db, sql, callback, 0, &zErrMsg);
	if( rc != SQLITE_OK ){
		fprintf(stderr, "SQL error: %s\n", zErrMsg);
		sqlite3_free(zErrMsg);
	}
	else{
		fprintf(stdout, "Records created successfully\n");
	}
}
/////////////////////     FIN DE LA PARTIE QUI POSE PROBLEME    ///////////////
 
// MAIN
 
int main()
{
	int tempInt;
	char *tempChar;
 
	tempInt=2;
	tempChar="test";
 
	openDb();
 
	SQLnewExercise(tempInt,tempChar);// C'EST APPAREMMENT QUAND LE PROGRAMME APPELLE CETTE FONCTION QUE J'AI UN SOUCI
 
	closeDb();
 
	return 0;
}

Merci d'avance pour vos réponses.

18/03/2014, 00h27
imperio
Deja cette ligne :
Code:

1 2 sql="INSERT INTO Exercises (Id,Name)"\ "VALUES (3, '%s');",Name;
est invalide en C. Si tu veux faire ca, utilise sprintf ou snprintf. Exemple :
Code:

1 2 3 char sql[250] = ""; snprintf(sql, 249, "INSERT INTO Exercises (Id,Name) VALUES (3, \"%s\");", Name);
Pour ce qui concerne les requetes de sqlite3 (si elle ne fonctionne toujours pas apres cette modification).

Votre code fonctionne bien uniquement si je met tout mon code dans la fonction main. Dès que je veux structurer mon code en fonctions, j'ai cette erreur :

Segmentation fault (core dumped)

Voici le morceau de code qui pose problème selon moi. Après avoir essayer un tas de choses, je ne sais plus vers quelle piste me diriger.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
 
 
////////////  PARTIE QUI POSE PROBLEME SELON MOI ///////////////
void SQLnewExercise(int Id,char *Name)
{
	sqlite3 *db;
	char *zErrMsg = 0;
	int rc;
	char sql[250] = "";
 
	/* Create SQL statement */
	   snprintf(sql, 249,"INSERT INTO COMPANY (ID,NAME,AGE,ADDRESS,SALARY) VALUES (6,\"%s\" , 25, 'Rich-Mond ', 65000.00 );",Name);
 
 
	/* Execute SQL statement */
	rc = sqlite3_exec(db, sql, callback, 0, &zErrMsg);
	if( rc != SQLITE_OK ){
		fprintf(stderr, "SQL error: %s\n", zErrMsg);
		sqlite3_free(zErrMsg);
	}
	else{
		fprintf(stdout, "Records created successfully\n");
	}
}
/////////////////////     FIN DE LA PARTIE QUI POSE PROBLEME    ///////////////
 
// MAIN
 
int main()
{
	int tempInt;
	char *tempChar;
 
	tempInt=2;
	tempChar="test";
 
	openDb();
 
	SQLnewExercise(tempInt,tempChar);// C'EST APPAREMMENT QUAND LE PROGRAMME APPELLE CETTE FONCTION QUE J'AI UN SOUCI
 
	closeDb();
 
	return 0;
}

18/03/2014, 02h43
Invité

Bonjour,

Je ne vois aucune initialisation du pointeur sqlite3 *db;.

Citation:

Envoyé par Winjerome

Bonjour,

Je ne vois aucune initialisation du pointeur sqlite3 *db;.

Bonjour, si je ne dis pas de bêtises *db envoi vers le nom de ma base de donnée après avoir utilisé cette fonction :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 void openDb() { sqlite3 *db=NULL; //char *zErrMsg = 0; int rc; /* Open database */ rc = sqlite3_open("test.db", &db); if(rc){ fprintf(stderr, "Can't open database: %s\n", sqlite3_errmsg(db)); exit(0); } else{ fprintf(stderr, "Opened database successfully\n"); } }

Sauf que
Code:

1 2 3 4 5 6 7 void openDb() { sqlite3 *db=NULL; rc = sqlite3_open("test.db", &db); ... }
et
Code:

1 2 3 void SQLnewExercise(int Id,char *Name) { sqlite3 *db;
Sont deux variables distinctes qui n'ont rien à voir entre elles, hormis leur nom.

18/03/2014, 06h26
Neckara
Bonjour,
Citation:
Envoyé par imperio

Deja cette ligne :

Code:

1 2 sql="INSERT INTO Exercises (Id,Name)"\ "VALUES (3, '%s');",Name;

est invalide en C. Si tu veux faire ca, utilise sprintf ou snprintf. Exemple :

Code:

1 2 3 char sql[250] = ""; snprintf(sql, 249, "INSERT INTO Exercises (Id,Name) VALUES (3, \"%s\");", Name);

Pour ce qui concerne les requetes de sqlite3 (si elle ne fonctionne toujours pas apres cette modification), regarde l'exemple sur cette page.
Attention aux injections SQL. sqlite n'a pas des requêtes préparées ?

Citation:

Envoyé par Winjerome

Sauf que

Code:

1
2
3
4
5
6
7
void openDb()
{
    sqlite3 *db=NULL;
    rc = sqlite3_open("test.db", &db);
 
    ...
}

Code:

1
2
3
void SQLnewExercise(int Id,char *Name)
{
	sqlite3 *db;

Sont deux variables distinctes qui n'ont rien à voir entre elles, hormis leur nom.

Exact. C'était bien la source de mon problème.

J'ai donc initialisé le pointeur à NULL dans ma fonction open.
Et j'ai rajouté un paramètre "db" dans ma fonction "SQLnewExercise".

Tout fonctionne à présent.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
 
void SQLnewExercise(int Id,char *Name,sqlite3 *db) //ajout de "sqlite3 *db" dans les paramètres
{
	//sqlite3 *db; // plus besoin de cette variable ici du coup
	char *zErrMsg = 0;
	int rc;
	char sql[250] = "";
 
	/* Create SQL statement */
	   snprintf(sql, 249,"INSERT INTO COMPANY (ID,NAME,AGE,ADDRESS,SALARY) VALUES (6,\"%s\" , 25, 'Rich-Mond ', 65000.00 );",Name);
 
 
	/* Execute SQL statement */
	rc = sqlite3_exec(db, sql, callback, 0, &zErrMsg);
	if( rc != SQLITE_OK ){
		fprintf(stderr, "SQL error: %s\n", zErrMsg);
		sqlite3_free(zErrMsg);
	}
	else{
		fprintf(stdout, "Records created successfully\n");
	}
}
 
// MAIN
 
int main()
{
        sqlite3 *db=NULL;
	char *zErrMsg = 0;
	int rc;
 
	/* Open database */
	rc = sqlite3_open("test.db", &db);//db prend la valeur "test.db"
	if(rc){
		fprintf(stderr, "Can't open database: %s\n", sqlite3_errmsg(db));
		exit(0);
	}
	else{
		fprintf(stderr, "Opened database successfully\n");
	}
 
 
	int tempInt;
	char *tempChar;
 
	tempInt=2;
	tempChar="test";
 
	SQLnewExercise(tempInt,tempChar,db);//on rajoute un paramètre db qui a en mémoire la bonne valeur
 
	closeDb();
 
	return 0;
}

Pour les questions de sécurité il n'y a aucun souci. C'est une petite application pour un usage personnelle. Et les informations ne sont absolument pas confidentielles.

Merci à tous pour votre aide.