[VALGRIND] Conditional jump or move depends on uninitialised value(s)

Bonjour,

en netoyant quelques unes de mes fonctions favorites, j'ai voulu optimiser un peu et debuger avec Valgrind ma petite librairie.

Mais la c'est le drame, je tombe donc sur un probleme mineur mais je vois pas quoi faire.

Le message de valgrind est celui-ci :
Conditional jump or move depends on uninitialised value(s).

Et voici mon code xD:

Code:

---

1 2 3 4 5 6 7 8 9 10 11

int wtablen(char **wtab) { int i;   i = 0; if (wtab) while (wtab[i]) i++; return (i); }

---

i est initialise, je ne pense pas aller trop loin dans le tableau...

Merci pour votre aide :)

Bonsoir,

Essaie

Code:

---

if (wtab==NULL)

---

… si c'est bien cette macro que tu utilises. A priori, NULL est résolue en ((void *)0) mais ce n'est pas une garantie. Cela dit, ce code est valide quand même valide, même si transtyper directement un pointeur en entier ou en booléen n'est pas très propre. Je ne sais pas pourquoi valgrind se plaint.

Citation:

---

Envoyé par blazed

Et voici mon code xD:

Code:

---

1 2 3 4 5 6 7

int wtablen(char **wtab) { …   return (i); }

---

---

« return » est un mot-clé, pas une fonction. Les parenthèses sont inutiles.

Le problème est sûrement ailleurs, peut-on avoir un code compilable reproduisant le problème ?

J'utilise moi aussi de plus en plus valgrind, et je me demande si avec les options -v pour valgrind lors de l'executon, et l'option -g pour gcc à la compilation, on ne peut pas avoir un message plus précis. Du genre, indication de la ligne

Sinon, mais je pense que j'ai tord,

Code:

---

1 2 3 4

int i = 0;   i = 0;

---

Expliciter la définition de i, lors de sa déclaration ...

Bonjour,

merci de toutes ces reponces, je n'ai toujours pas trouver de solution pour resoudre mon probleme. Je reposterais dans ce topic si je trouve au file des programmes :)

Pour les parentheses sur le retrun c'est parce que j'aime bien ^^

J'utilise ces flags pour valgrind :
valgrind --leak-check=yes --show-reachable=yes

Merci encore et au plaisir.

Citation:

---

Envoyé par blazed

Pour les parentheses sur le retrun c'est parce que j'aime bien ^^

---

Tu fais comme tu veux, mais tu te forces ainsi à prendre une mauvaise habitude. Ce n'est pas forcément ce qu'il y a de plus sensé. Cela risque de te conduire à terme à des effets de bords de ceux que tu rencontres aujourd'hui.

Et puis peut être que vos collègues ne vont pas apprécié ( compagnie ). Du coup, vous allez vite changé. ( Surtout qu'un gars comme moi, il va vous le dire de changer, et risque de corriger ... )

Citation:

---

Le message de valgrind est celui-ci :
Conditional jump or move depends on uninitialised value(s).

---

Ce message ne peut-il pas provenir du fait que l'argument x correspondant au paramètre wtab de la fonction n'est pas initialisé au moment de l'appel ?
Du genre :

Code:

---

1 2 3

char ** x; .... wtablen(x);

---

bien que, dans ce cas, l'erreur devrait être signalée sur la ligne de l'appel (mais c'est peut-être lié à un niveau d'optimisation du code ?).

diogene, merci pour l'idée je vais regarder dans ce sens car en effet j'utilise une moulinette de test pour différent cas de figure et cela pourrait provenir des tests.

Mais ne vous inquiétez pas, si je devais travailler avec vous je changerais mon mode opératoire sans problème ;) (et puis même si j'aime bien, c'est dans ma norme :p )