Comparaison pointeur entier

**kathlyn222** · 10/12/2008, 09h59

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 :

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

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 !

**Médinoc** · 10/12/2008, 10h26

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

**Emmanuel Delahaye** · 10/12/2008, 10h28

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 :

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

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...

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

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

**kathlyn222** · 10/12/2008, 12h58

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 !

**Médinoc** · 10/12/2008, 13h01

Rendons à César ce qui appartient à César: C'est Emmanuel qui a corrigé l'erreur, pas moi...

**Emmanuel Delahaye** · 10/12/2008, 13h03

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.

**kathlyn222** · 10/12/2008, 13h16

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 !