probléme de mémoire (double free).

Bonjour ,

J'ai un problème assez étrange .

il me met double free qu'a partir du 9 , fichier importimage.c : 207

Si je l'éxecute avec valgrind aucun problème .

voici les sources :
- main.c : ici .
- importimage.c : ici
- importimage.h :ici


Merci d'avance .

nb : j'ai hésité a le poster sur le forum gtk mais le problème ne semble pas venir de gtk+ .

ouhhhh beaucoup de choses...

J'ai pas regardé le main, mais juste dans importimage :

Code:

---

1 2 3 4

if (stream == NULL) return -1; struct StructMemoire *mem = stream;

---

Pas C89/90. Pas d'instructions AVANT des déclarations.

Code:

---

1 2 3 4

mem->memoire = realloc(mem->memoire, realsize + mem->taille + 1); if (mem->memoire == NULL) return -2;

---

Pas de pointeur intermédiaire. Si allocation échoue, impossible de libérer ce qui était déjà alloué.

Code:

---

1 2	mem->memoire[mem->taille] = 0;

---

Pas forcément la meilleur manière de mettre à NULL..

Code:

---

1 2 3

tab = malloc(12 * sizeof (*tab)); tab2 = malloc(12 * sizeof (*tab2));

---

Pas de vérifications que les alloc se soeitn bien passées.

Code:

---

1 2 3 4 5 6

for(i=0;i<12;i++) { tab[i] = malloc(80 * sizeof (char)); tab2[i] = malloc(80 * sizeof (char)); }

---

idem

Code:

---

1 2 3 4 5 6

for(i=0;i<12;i++) { fprintf(stderr,"liberation : %d\n",i); fprintf(stderr,"liberation : %p\n",tab2[i]); free(tab2[i]);

---

idem

Code:

---

1 2 3

free(tab2); tab2 =NULL ;

---

pas dans le if ( tab2 != NULL )

Citation:

---

Envoyé par souviron34

ouhhhh beaucoup de choses...

J'ai pas regardé le main, mais juste dans importimage :

Code:

---

1 2 3 4

if (stream == NULL) return -1; struct StructMemoire *mem = stream;

---

Pas C89/90. Pas d'instructions AVANT des déclarations.

Code:

---

1 2 3 4

mem->memoire = realloc(mem->memoire, realsize + mem->taille + 1); if (mem->memoire == NULL) return -2;

---

Pas de pointeur intermédiaire. Si allocation échoue, impossible de libérer ce qui était déjà alloué.

Code:

---

1 2	mem->memoire[mem->taille] = 0;

---

Pas forcément la meilleur manière de mettre à NULL..

---

ce code n'est pas de moi mais des développeurs de libcurl : ici

Mais c'est corrigé .

Citation:

---

Envoyé par souviron34

Code:

---

1 2 3

tab = malloc(12 * sizeof (*tab)); tab2 = malloc(12 * sizeof (*tab2));

---

Pas de vérifications que les alloc se soeitn bien passées.

Code:

---

1 2 3 4 5 6

for(i=0;i<12;i++) { tab[i] = malloc(80 * sizeof (char)); tab2[i] = malloc(80 * sizeof (char)); }

---

idem

Code:

---

1 2 3 4 5 6

for(i=0;i<12;i++) { fprintf(stderr,"liberation : %d\n",i); fprintf(stderr,"liberation : %p\n",tab2[i]); free(tab2[i]);

---

idem

Code:

---

1 2 3

free(tab2); tab2 =NULL ;

---

pas dans le if ( tab2 != NULL )

---

Corrigé



Mais cela ne résout pas mon problème , j'ai édité les liens .

d'abord le truc du realloc n'est pas corrigé..

Ensuite je ne connais pas libcurl, mais quand je vois :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

curl_easy_setopt(curl,CURLOPT_WRITEDATA,(void *) &memoire_s); .... curl_easy_perform(curl); ....   free(begin); memoire_s.memoire = NULL;   .... curl_easy_cleanup(curl);

---

Que fait easy_setopt ? ça alloue la mémoire ?
Que fait easy_cleanup ? ça désalloue pas la mémoire ?
Si oui, alors ne pas faire free(begin)....

Citation:

---

Envoyé par souviron34

d'abord le truc du realloc n'est pas corrigé..


Que fait easy_setopt ? ça alloue la mémoire ?
Que fait easy_cleanup ? ça désalloue pas la mémoire ?
Si oui, alors ne pas faire free(begin)....

---

1) Non cela ajoute une donné au callback (memorycallback).
2) Ouai celle allloué par curl_easy_init .
3) begin n'as aucun rapport avec libcurl .



J'ai remarqué que le problème vient de :

Code:

---

1 2 3 4 5 6 7 8

for(i=0;i<12;i++) { fprintf(stderr,"liberation : %d\n",i); fprintf(stderr,"liberation : %p\n",tab2[i]); free(tab2[i]); tab2[i]=NULL ;   }

---

au bout de la 9ème itération : baam


liberation : 9
liberation : 0x5e67c0
*** glibc detected *** ./importimage: double free or corruption (!prev): 0x00000000005e67c0 ***



EDIT : Une chaine beaucoup trop longue a déclenché une corruption de la mémoire.
.


Merci de ton aide souviron , je vais m'occuper de ce realloc .