Comparaison pointeur entier

Version imprimable

Bonjour,

J'ai un projet de communication entre une interface, un administrateur et plusieurs machines a effectuer.

Deux soucis se présentent.

D'abord quand je compile, le warning suivant :
"warning: comparison between pointer and integer"

Qui se dirige vers la bout de code suivant :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 while((entree = readdir(rep)) != NULL) //Tant qu'il reste quelque chose a lire { if (strncmp(entree->d_name, ".", 1) == 0) continue; if (entree->d_name == DT_REG || entree->d_type == DT_DIR) // Ne lire que les fichiers et les repertoires { *length = *length + strlen(entree->d_name) +1; // Modifie la taille des donnees newcontenu = realloc(contenu, *length); //Reallocation memoire if (newcontenu == NULL) { perror("realloc"); // Si erreur dans realloc *length = 0; return NULL; } char *retval; retval = NULL; contenu = newcontenu; strncat(contenu, entree->d_name, NAME_MAX); // Ajoute le nom strcat(retval,"\n"); // Ajoute le retour a la ligne } }
C'est un bout d'une fonction fournie par les profs, et je n'en comprends pas tous les éléments !

Deuxieme souci, j'ai un superbe sgmentation fault qui me poursuit des que j'execute mon programme.
J'ai beau rajouter des malloc partout ou j'avais pu les oublier, il persiste !

J'espere que ce probleme est lié au premier, et que vous pourrez m'aider !

Merci beaucoup !

10/12/2008, 10h26
Médinoc

Il manque la déclaration des différentes variables.
Poste un code minimal mais compilable qui reproduit l'erreur, s'il te plait.

Citation:

Envoyé par kathlyn222

D'abord quand je compile, le warning suivant :
"warning: comparison between pointer and integer"

Qui se dirige vers la bout de code suivant :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
  while((entree = readdir(rep)) != NULL) //Tant qu'il reste quelque chose a lire
    {
/* -ed- les entrées de fichiers commençant par "." sont ignores */
      if (strncmp(entree->d_name, ".", 1) == 0)
	continue;
 
/* -ed- c'est probablement ici qu'il y a l'avertissement. 
 
entree->d_name == DT_REG
 
n'a aucun sens. DT_REG n'est pas un pointeur. 
Le bon code est probablement : 
 
entree->d_type == DT_REG
 
*/
      if (entree->d_name == DT_REG || entree->d_type == DT_DIR) // Ne lire que les fichiers et les repertoires
 
	{
	  *length = *length + strlen(entree->d_name) +1; // Modifie la taille des donnees
	  newcontenu = realloc(contenu, *length); //Reallocation memoire
	  if (newcontenu == NULL)
	    {
	      perror("realloc"); // Si erreur dans realloc
	      *length = 0;
	      return NULL;
	    }
 
      char *retval;
      retval = NULL;
      contenu = newcontenu;
      strncat(contenu, entree->d_name, NAME_MAX); // Ajoute le nom
      strcat(retval,"\n"); // Ajoute le retour a la ligne
 
/* -ed- ce dernier bloc d'instruction n'a aucun sens. 
   Si retval vaut NULL, son usage dans strncat() provoque un 
   comportement indéfini.
*/
	}
    }

C'est un bout d'une fonction fournie par les profs,

Si c'est vrai, change d'école, et vite...

Citation:

et je n'en comprends pas tous les éléments !

J'ai ajouté quelques commentaires... (-ed-)

10/12/2008, 12h58
kathlyn222

warning reglé

Merci pour ton aide Medinoc, avec la correction de cette erreur (j'espere erreur de frappe de la part du prof), le warning n'apparait plus.

Quant au segmentation faults, il doit venir de bien plus loin...
Je continue de chercher !
10/12/2008, 13h01
Médinoc

Rendons à César ce qui appartient à César: C'est Emmanuel qui a corrigé l'erreur, pas moi... ;)
10/12/2008, 13h03
Emmanuel Delahaye

Citation:

Envoyé par kathlyn222

Quant au segmentation faults, il doit venir de bien plus loin...
Je continue de chercher !

Je l'ai déjà indiqué. retval pointe sur NULL. Si tu écris en NULL, le comportement est indéfini.
10/12/2008, 13h16
kathlyn222

Excuse-moi pour l'erreur, merci Emmanuel !

Et pour retval = NULL, j'ai reglé le problème en le définissant autrement.

Le segmentation fault doit venir d'une autre fonction, mais j'ai l'impression de chercher une aiguille dans une botte de foin.

Je suppose que cela doit venir d'un malloc oublié ou autre erreur de ce type, mais je ne la vois pas !
10/12/2008, 21h06
Thierry Chappuis

Citation:

Envoyé par kathlyn222

Excuse-moi pour l'erreur, merci Emmanuel !

Et pour retval = NULL, j'ai reglé le problème en le définissant autrement.

Que veux-tu dire par "en le définissant autrement" ?

Thierry
11/12/2008, 12h23
oniric

essaye d'utiliser un debuggeur comme gdb si tu es sous unix. Il te permettra de trouver facilement ou se produit le segfault et donc d'identifier plus facilement la cause.
11/12/2008, 13h50
Thierry Chappuis

Citation:

Envoyé par oniric

essaye d'utiliser un debuggeur comme gdb si tu es sous unix. Il te permettra de trouver facilement ou se produit le segfault et donc d'identifier plus facilement la cause.

gdb est utilisable sous Windows également.

Thierry