Discussion :

[kreel]

bonjour tout le monde,

Je suis un peu embêté avec un problème d'allocatin mémoire pour un tableau à 2 dimensions. En fait le problème n'est pas sur la façon d'allouer de la mémoire pour le tableau, ça je pense savoir faire, même si vous confirmer au cas où. Le problème viens du fait qu'au bout d'un moment dans mon programme certaines cases de ce tableau sont écrasés sans que je le demande.
Pour plus de précision, je n'ai pas effectué de libération de la mémoire sur ce tableau.
voici le code :

char **bots; //déclarer en global bien sur
bots = (char **)malloc(size * sizeof(*bots)); //pour l'allocation mémoire de la 1ère dimension
.....
bots[i] = malloc(sizeof(char)*strlen(tmp)); //pour la 2ème dimension
//l'allocation (2è dimension) se fait au fur et à mesure dans une boucle while

et maintenant l'affichage que j'obtiens :

//lors du premier appel à une fonction
bots[0] = googlebot
bots[1] = mediapartners-google
bots[2] = googlebot
bots[3] = mediapartners-google
bots[4] = msnbot
bots[5] = voilabot
bots[6] = gigabot
bots[7] = seekbot
bots[8] = newsgator
bots[9] = snapbot
bots[10] = ebiness
bots[11] = biglotron
bots[12] = henrilerobotmirago
************************************
************************************
//lors du second appel à cette meme fonction
bots[0] = lj611675.inktomisearch.com
bots[0] = lj611675.inktomisearch.com
bots[0] = lj611675.inktomisearch.com
bots[0] = lj611675.inktomisearch.com
bots[0] = lj611675.inktomisearch.com

Pour précision, la fonction appelée ne modifie pas le tableau, il ne fait que le consulter. Hors, entre 2 consultations le compilateur s'est arrangé pour modifier les valeurs aux adresses allouées pour le tableau.
Comment puis-je faire pour être sûr que le bloc mémoire du tableau ne soit pas modifié ?

Merci d'avance à tous.

[souviron34]

déjà :

Code :

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

1
2
bots[i] = malloc(sizeof(char)*(strlen(tmp)+1)); //pour la 2ème dimension

Ensuite comment fais-tu la copie de la chaîne ?

j'espère que c'est bien :

Code :

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

1
2
 
strcpy ( bots[i], tmp );

[kreel]

Oui c'est bien de cette façon que je copie la chaine.
C'est grave docteur ? sinon je vais rajouter le +1, même si je pense que ça ne changera pas mon problème mais permettra d'être plus juste.
Alors une solution ? ;-)

[Thierry Chappuis]

Oui c'est bien de cette façon que je copie la chaine.
C'est grave docteur ? sinon je vais rajouter le +1, même si je pense que ça ne changera pas mon problème mais permettra d'être plus juste.
Alors une solution ? ;-)

Le +1 est très important ici (obligatoire), car il te permet de réserver une case pour le caractère nul de fin de chaine.

Thierry

[kreel]

Donc après la modification conseillé, toujours le problème. J'ai vérifié l'adresse du pointeur du tableau et elle ne change pas. Donc lors de l'éxécution le compilateur réécris bien "par dessus" le tableau.
Je vais essayer de débugger pour voir où est fait l'écrasement. Cependant si quelqu'un à une suggestion, je suis preneur.

[souviron34]

mets le code incriminé...

[kreel]

J'aimerais beaucoup mais cela peut venir d'un peu partout dans le programme. Je participe en fait à une évolution d'un logiciel libre qui s'appel webdruid. Ce qui inclue pas mal de ligne de code.
Dés que j'ai localisé d'où vient le problème et si je ne sais toujours pas comment le résoudre malgré tout, j'afficherais le code incriminé.
En tout cas merci beaucoup de m'aider.

[souviron34]

tu es sur *n*x ?

utilise ddd...

[Emmanuel Delahaye]

Je suis un peu embêté avec un problème d'allocatin mémoire pour un tableau à 2 dimensions. En fait le problème n'est pas sur la façon d'allouer de la mémoire pour le tableau, ça je pense savoir faire, même si vous confirmer au cas où. Le problème viens du fait qu'au bout d'un moment dans mon programme certaines cases de ce tableau sont écrasés sans que je le demande.
Pour plus de précision, je n'ai pas effectué de libération de la mémoire sur ce tableau.
voici le code :

Pour la deuxième dimension, pourquoi tu n'utilises pas strdup() au moment voulu ? C'est POSIX.1, donc très portable, et il n'y a pas d'erreur de taille !

[kreel]

pour répondre à souviron, oui je suis sous linux (tout de meme ;-)) et j'utilise valgrind là. enfin j'essaie, pour le moment je tatone.
En ce qui concerne strdup, je vais tenter sur la 2ème dimension pour voir. J'avais essayer sur la 1ère et ça n'avais pas marcher du coup j'ai pas pensé à l'utiliser sur la 2ème.

[kreel]

C'est bon trouvé, le précédent programmeur avait laissé un free mal placé. Tout ça pour ça . En tout cas merci pour tout.