Discussion :

[zouari.rami]

Bonjour, j'ai voulu declarer deux tableau dynamique de char ( chaine de caractere) dont je ne sais pas la taille j'ai procédé de la facon suivante:

Code :

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

1
2
3
 
char* chaine;
char* chaine1;

et j'ai inséré dans une boucle while le coe suivant:

Code :

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

1
2
3
4
5
6
7
 
if (k==0) chaine= malloc (sizeof(char));
else chaine = realloc (chaine,sizeof(chaine)+sizeof(char));
chaine[k]=chain[i];  k++;
if (l==0) chaine1= malloc (sizeof(char));
else chaine1 = realloc (chaine1,sizeof(chaine1)+sizeof(char));
chaine1[l]=chain[i]; l++;

mé lors de la compilation une erreur de ce type surgit:

Code :

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

1
2
3
4
5
6
7
8
9
 
projet1(6875) malloc: *** error for object 0x100554: incorrect checksum for freed object - object was probably modified after being freed.
*** set a breakpoint in malloc_error_break to debug
projet1(6875) malloc: *** error for object 0x100564: incorrect checksum for freed object - object was probably modified after being freed.
*** set a breakpoint in malloc_error_break to debug
projet1(6875) malloc: *** error for object 0x100574: incorrect checksum for freed object - object was probably modified after being freed.
*** set a breakpoint in malloc_error_break to debug
projet1(6875) malloc: *** error for object 0x100570: incorrect checksum for freed object - object was probably modified after being freed.
*** set a breakpoint in malloc_error_break to debug

alors j'étais sur qu'il y a un probleme avec l'allocation que j'ai fait
merci de me repondre

[gl]

Il y a deux problèmes dans l'utilisation de realloc :

• sizeof(chaine) retourne la taille du pointeur (qui est donc la même à chaque appel), pas la taille allouée. Ainsi realloc va systématiquement allouer la même chose. Ton erreur vient probablement de là : tu alloues moins que ce que tu penses et tu vas ensuite écrire en dehors de la zone allouée.
• Pour utiliser correctement realloc, il ne faut pas directement récupérer la valeur de retour dans le pointeur final. En effet, en cas d'erreur realloc retourne NULL, ici tu aurais donc le pointeur chaine qui vaut NULL et plus aucun lien sur la mémoire précédemment allouée qui ne pourrait donc pas être libérée ==> fuite mémoire.

Sinon deux petites remarques complémentaires:

• Il est possible d'utiliser realloc(NULL, ...) qui se comporte alors comme malloc (ce qui devrait te permettre de simplifier ton code).
• Il est très inefficace de réallouer un octet de plus à chaque fois. Sauf contrainte très particulière, on préfère faire croître la zone allouée de manière plus importante à chaque realloc() et faire appel à realloc() moins fréquemment. Quitte à réduire au strict nécessaire en fin de traitement.

[zouari.rami]

Merci bcp de ta reponse elle m'a beaucoup aidé surtout le sizeof(chaine) qui retourne la taille du pointeur et non la taille du contenu.