Discussion :

[Magnum8760]

Bonjour, gcc m'indique *** stack smashing detected ***: <unknown> terminated avec cette fonction.
J'alloue surement pas assez de mémoire pour ma variable newFunction mais je ne vois pas comment améliorer tout ça (à cet heure) .

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
/**
* affichage des informations de l'écran GL.
* @param chemin du fichier.csv
*/
void windowFunction(char* sourcesPath){
			int val;
			char *newFunction = NULL;
			char 	argument[40],
			 	description[100];
			FILE *file = NULL;
	                file = fopen(sourcesPath, "r"); /* read file */
	                if (file == NULL) 
                                 exit(1);
			else {
				do{
				fscanf(file, "%s %s", argument, description);
							newFunction = (char*)malloc(sizeof(char));
							//strcpy(newFunction, argument);
							printf("salut la new funct est %s \n", newFunction);
							sprintf(newFunction, "%s", argument);
							val = glutGet(newFunction);
							printf("%d\n", val);
				}while(!feof(file)); // tester la fin d'un fichier*/
				fclose(file); /* fermer un fichier ouvert */
			}
                        free(newFunction);
}

[edgarjacobs]

Hello,

Ligne 17: tu alloues 1 octet pour newFuction, que tu n'initialises pas.
Ligne 19: tu fais un printf() de newFunction, mais il n'y a peut-être de pas de \0 dans le string
Ligne 20: il n'y a sans doute pas assez de place pour copier argument dans newFunction

[Médinoc]

Aussi, es-tu 100% sûr que les lignes de ton fichier en dépassent pas les 40 et 100 caractères?

[Magnum8760]

Bonjour, voici quelques une des lignes de mon fichiers. Les fonctions ne dépasse pas les 40 caractères enfin max 30 :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
GLUT_WINDOW_ACCUM_RED_SIZE Number_of_bits_of_red_stored_in_the_current_window's_accumulation_buffer._Zero_if_the_window_is_color_index
GLUT_WINDOW_ACCUM_GREEN_SIZE Number_of_bits_of_green_stored_in_the_current_window's_accumulation_buffer._Zero_if_the_window_is_color_index
GLUT_WINDOW_ACCUM_BLUE_SIZE Number_of_bits_of_blue_stored_in_the_current_window's_accumulation_buffer._Zero_if_the_window_is_color_index
GLUT_WINDOW_ACCUM_ALPHA_SIZE Number_of_bits_of_alpha_stored_in_the_current_window's_accumulation_buffer._Zero_if_the_window_is_color_index

J'avais complètement oublié cette partie. En fait, j'avais pas très bien compris pourquoi disais t'on que j'allouais seulement un octet ici :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
char *newFunction = NULL;
...
newFunction = (char*)malloc(sizeof(char));

J'initialise bien à NULL un pointeur vers un tableau de futur caractères.

Ok ok j'alloue à la taille d'un char, 1 mot, 1 octet ... mais je ne sais pas comment m'y prendre pour allouer à la bonne taille...

[Magnum8760]

Je ne serrais pas le faire pour le moment mais je vais essayer ceci tout à l'heure

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
#define MAX 200
 
char *newFunction = NULL;
...
			char 	argument[MAX],
			 	description[MAX];
...
newFunction = (char*)malloc(MAX * sizeof(char));

[Bousk]

Pourquoi newFunction aurait besoin d'une allocation dynamique, à fortiori de taille fixe, quand description et argument sont statiques ?