problème de realloc()

Version imprimable

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:

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:

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 ? :bug:

Citation:

Code:

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

Ca serait pas plutot ?

Code:

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

24/02/2007, 01h48
souviron34
2 erreurs majeures :

1) Count démarre à 0 :
Code:

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:

1 2 instruction_tab[count] = (char*)malloc(strlen(token)+1);
Et enfin comment est défini instruction_tab ??
26/02/2007, 17h25
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
26/02/2007, 17h34
souviron34
Citation:
Envoyé par Médinoc

On ne connait pas le type de token, mais si c'est un char* aussi, tu es mal barré.
:lol: aujoudh'ui tu tires plus vite que ton ombre...

mais il a fait appel a strtok donc c'est un char *... ;)
27/02/2007, 13h15
daweed

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

A plus et merci encore :yaisse2:

Bonjour,

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

le code qui contient le realloc :

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
 
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:

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:

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 :help:

01/03/2007, 22h35
Emmanuel Delahaye

Citation:

Envoyé par daweed

le code qui contient le realloc :

Ca ne compile pas...
02/03/2007, 00h32
souviron34
même erreur, même punition !!! :aie:
Code:

1 2 tabvar[nbevar].name = (char *)malloc(strlen(name)+1);
et aussi tu ne fais pas l'allocation pour value .... 8O
Code:

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

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

petite rectification :

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
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.

il te manque toujours le
Code:

1 2 tabvar[nbevar].name = (char *)malloc(strlen(name) +1 );
et pareil pour :
Code:

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

02/03/2007, 13h54
Franck.H
Code:

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:

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