Discussion :

[daweed]

Bonjour,

Je m'arrache les cheveux depuis plusieurs heures car je ne parvien pas à résoudre mon problème :

Voici la sortie de mon programme :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
*** glibc detected *** ./irawk: realloc(): invalid next size: 0x0804b0a8 ***
======= Backtrace: =========
/lib/tls/i686/cmov/libc.so.6[0xb7e5d3aa]
/lib/tls/i686/cmov/libc.so.6(__libc_realloc+0xff)[0xb7e5dcdf]
./irawk[0x80487cd]
./irawk[0x8048d2a]
./irawk[0x8048fab]
/lib/tls/i686/cmov/libc.so.6(__libc_start_main+0xdc)[0xb7e0a8cc]
./irawk[0x8048591]

Je pense que l'erreur vien de ce code la :

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
 
for (str = instruction, count = 0 ; ; str = NULL) {
      token = strtok_r(str, "\"", &saveptr);
      if (token == NULL)
            break;
       printf("token : %s\n", token);
       instruction_tab = (char **)realloc(instruction_tab, (sizeof(char*)));
       if(instruction_tab == NULL){
       		printf("erreur d'allocation mémoire\n");
       		return NULL;
       }
       instruction_tab[count] = (char*)malloc(sizeof(token));
       if(instruction_tab[count] == NULL){
       printf("erreur d'allocation mémoire\n");
       		for(i = 0; i < count ; i++){
       			free(instruction_tab[i]);
       		}
       		free(instruction_tab);
       		free (instruction);
		return NULL;
	}
	strcpy(instruction_tab[count], token);
        count ++;
 }

Est-ce que vous voyez une erreur?
Une idée ?

[Sanguko]

Code :

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

1
2
 
       instruction_tab = (char **)realloc(instruction_tab, (sizeof(char*)));

Ca serait pas plutot ?

Code :

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

1
2
 
       instruction_tab = (char **)realloc(instruction_tab, (sizeof(char*))*count);

[souviron34]

2 erreurs majeures :

1) Count démarre à 0 :

Code :

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

1
2
       instruction_tab = (char **)realloc(instruction_tab, (sizeof(char*)*(count+1)));

2) une chaîne doit être allouée avec la place pour le caractère de fin '\0' :

Code :

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

1
2
       instruction_tab[count] = (char*)malloc(strlen(token)+1);

Et enfin comment est défini instruction_tab ??

[Médinoc]

1. On ne connait pas le type de token, mais si c'est un char* aussi, tu es mal barré.
2. Pas de cast du retour de malloc() en C.
3. Mauvais usage de realloc(). Voir ici : http://emmanuel-delahaye.developpez....es.htm#realloc

[souviron34]

1. On ne connait pas le type de token, mais si c'est un char* aussi, tu es mal barré.

aujoudh'ui tu tires plus vite que ton ombre...

mais il a fait appel a strtok donc c'est un char *...

[daweed]

Merci beaucoup souviron, ça marche tout de suite mieux comme ça

A plus et merci encore

[daweed]

Bonjour,

Nouveau problème de realloc()!! décidément cette fonction ne m'aime pas!

le code qui contient le realloc :

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
28
29
30
31
32
33
 
nbevar++;
tabvar = (variable *)realloc(tabvar, (nbevar * sizeof(variable)));
if (tabvar == NULL){
	printf("erreur d'allocation memoire\n");
	exit(-1);
}
//on nomme la variable
tabvar[nbevar].name = (char *)malloc(strlen(name));
if (tabvar[nbevar].name == NULL){
	printf("erreur d'allocation memoire\n");
	free(tabvar+ nbevar);
	nbevar--;
	free_tabvar();
	exit(-1);
}
strcpy(tabvar[nbevar].name, name);
if(is_num(val) == 1){// si c'est un nombre
	tabvar[nbevar].is_int = 1;
	tabvar[nbevar].val = atoi(val);
	strcpy(tabvar[nbevar].name, name);
}else{// si c'est une chaîne
	tabvar[nbevar].is_int = 0;
	tabvar[nbevar].value = (char *)malloc(strlen(val));
	if (tabvar[nbevar].value == NULL){
		printf("erreur d'allocation memoire\n");
		free(tabvar+ nbevar);
	        nbevar--;
                free_tabvar();
                exit(-1);
       }
}
strcpy(tabvar[nbevar].value, val);

ma structure variable :

Code :

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

1
2
3
4
5
6
7
 
typedef struct _variable{
	char * value;
	char * name;
	int  val;
	int is_int;	//0 ou 1
}variable;

l'erreur retournée :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
*** glibc detected *** ./irawk: realloc(): invalid next size: 0x0804c0a0 ***
======= Backtrace: =========
/lib/tls/i686/cmov/libc.so.6[0xb7e503aa]
/lib/tls/i686/cmov/libc.so.6(__libc_realloc+0xff)[0xb7e50cdf]
./irawk[0x804886c]
./irawk[0x804a206]
./irawk[0x804a467]
./irawk[0x804a6eb]
/lib/tls/i686/cmov/libc.so.6(__libc_start_main+0xdc)[0xb7dfd8cc]
./irawk[0x80485f1]
======= Memory map: ========
etc.......
etc.....

Un nouveau Help à vous messieurs.... please

[Emmanuel Delahaye]

le code qui contient le realloc :

Ca ne compile pas...

[souviron34]

même erreur, même punition !!!

Code :

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

1
2
tabvar[nbevar].name = (char *)malloc(strlen(name)+1);

et aussi tu ne fais pas l'allocation pour value ....

Code :

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

1
2
strcpy(tabvar[nbevar].value, val);  // allouée ou ???

[daweed]

Pour info ça compile très bien, je n'ai pas mis tout mon code pour vous épargner.

petite rectification :

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
28
29
30
31
32
nbevar++;
tabvar = (variable *)realloc(tabvar, (nbevar * sizeof(variable)));
if (tabvar == NULL){
	printf("erreur d'allocation memoire\n");
	exit(-1);
}
//on nomme la variable
tabvar[nbevar].name = (char *)malloc(strlen(name));
if (tabvar[nbevar].name == NULL){
	printf("erreur d'allocation memoire\n");
	free(tabvar+ nbevar);
	nbevar--;
	free_tabvar();
	exit(-1);
}
strcpy(tabvar[nbevar].name, name);
if(is_num(val) == 1){// si c'est un nombre
	tabvar[nbevar].is_int = 1;
	tabvar[nbevar].val = atoi(val);
	strcpy(tabvar[nbevar].name, name);
}else{// si c'est une chaîne
	tabvar[nbevar].is_int = 0;
	tabvar[nbevar].value = (char *)malloc(strlen(val));
	if (tabvar[nbevar].value == NULL){
		printf("erreur d'allocation memoire\n");
		free(tabvar+ nbevar);
	        nbevar--;
                free_tabvar();
                exit(-1);
       }
        strcpy(tabvar[nbevar].value, val);
}

Du coup value est bien allouée.

name et val sont deux paramêtres passés à la fonction.
tabvar et nbevar sont des variables globales.

Le problème vien du réalloc, pour moi.

[souviron34]

il te manque toujours le

Code :

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

1
2
tabvar[nbevar].name = (char *)malloc(strlen(name) +1 );

et pareil pour :

Code :

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

1
2
	tabvar[nbevar].value = (char *)malloc(strlen(val)+1 );

[Franck.H]

Code :

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

tabvar = (variable *)realloc(tabvar, (nbevar * sizeof(variable)));

En générale il est préférable d'utiliser un pointeur temporaire du même type bien entendu soit une realloc correct et digne de ce nom:

Code :

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

1
2
3
4
5
6
7
8
9
 
...
type * tmp = realloc (tabvar, nbvar * sizeof (* tabvar));
 
if (tmp != NULL)
{
   tabvar = tmp;
}
...