Discussion :

[nzo70]

Bonjour j'ai actuellement une erreur de segmentation lorsque je fais un realloc sur une de mes structures, pourtant je cree bien cette structure dynamiquement avec un malloc

Code :

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

1
2
3
 
 
TypVoisins **lstCC = (TypVoisins **) malloc(tailleMax * sizeof(TypVoisins*));

ensuite je fais des realloc au cours de l'execution du programme et des le premier j'ai un 'segmentation fault' pourtant je ne vois pas pourquoi ça coince

Code :

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

1
2
3
 
 
lstCC = (TypVoisins **)realloc(lstCC, (sizeof(lstCC)+1) * sizeof(TypVoisins *));

ps : je sais que le cast sur un realloc n'est pas recommandé mais meme sans le cast ça ne passe pas

Est ce que quelqu'un voit d'ou sa peut venir?

merci

[Obsidian]

Code :

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

lstCC = (TypVoisins **)realloc(lstCC, (sizeof(lstCC)+1) * sizeof(TypVoisins *));

« lstCC » est un pointeur. « sizeof (lstCC) » vaut donc 4 sur une machine 32 bits, quelque soit la taille allouée de la zone mémoire qu'il pointe.

[jeroman]

Code :

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

lstCC = (TypVoisins **)realloc(lstCC, (sizeof(lstCC)+1) * sizeof(TypVoisins *));

Je ne comprends pas ce que tu cherches à faire ici. Tu veux allouer une taille de mémoire correspondant à la taille d'un pointeur, incrémentée de 1, multipliée par la taille d'un autre pointeur, ce qui fait (4+1)*4. Quelle en est la signification ?

Deuxièmement, ce code est susceptible de provoquer une fuite de mémoire. Il faut stocker le retour de realloc dans un pointeur temporaire, car si la fonction échoue, elle renvoie NULL. Or, là, tu écrases lstCC directement, sans chercher à savoir si realloc a réussi ou non. Si la fonction ne renvoie pas NULL, c'est qu'elle n'a pas échoué ; là, tu peux alors écraser l'ancienne valeur.

Le cast void* vers type_quelconque*, et vice versa, est implicite en C. Cela ne sert donc à rien de caster.

[Obsidian]

Code :

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

lstCC = (TypVoisins **)realloc(lstCC, (sizeof(lstCC)+1) * sizeof(TypVoisins *));

Je ne comprends pas ce que tu cherches à faire ici. Tu veux allouer une taille de mémoire correspondant à la taille d'un pointeur, incrémentée de 1, multipliée par la taille d'un autre pointeur, ce qui fait (4+1)*4. Quelle en est la signification ?

Je pense qu'il veut agrandir la zone mémoire qu'il a initialement alloué et qu'il pense qu'un sizeof() sur le pointeur qui pointe cette zone lui renverra sa taille.

[nzo70]

effectivement je pensais que le sizeof me renvoyait la taille de la zone memoire et le probleme venait effectivement de la...merci

[Obsidian]

De rien, mais n'oublie pas …

[nzo70]

j'aurais juste une autre question... en fait lorsque je fais appel a une fonction ,qui est dans le meme fichier que la fonction appelante, si la fonction appelée n'est pas ecrite avant la fonction appelante j'ai un warning et je suis obligé de mettre la fonction appelée dans un .h est ce normal?