Pb lors de la compilation sur appel ftw()

**slm59** · 08/10/2009, 14h28

Bonjour à tous,

Je préfère prévenir de suite : j'ai quelques notions en langage C mais sans plus.
Je présente mon problème : suite à un changement de machine Unix, un programme permettant de faire des stats ne fonctionne plus correctement. Il a été développé il y a quelques années.
J'ai tenté de le recompiler sur la nouvelle machine et j'ai ce message d'erreur (warning) :

"statsMens.c", line 315: warning: argument #3 is incompatible with prototype:
prototype: pointer to function(pointer to const char, pointer to const struct stat {unsigned long st_dev, array[3] of long st_pad1, unsigned long st_ino, unsigned long st_mode, unsigned long st_nlink, long st_uid, long st_gid, unsigned long st_rdev, array[2] of long st_pad2, long st_size, long st_pad3, struct timespec {..} st_atim, struct timespec {..} st_mtim, struct timespec {..} st_ctim, long st_blksize, long st_blocks, array[16] of char st_fstype, array[8] of long st_pad4}, int) returning int : "ftw.h", line 107
argument : pointer to function(pointer to char, pointer to struct stat {unsigned long st_dev, array[3] of long st_pad1, unsigned long st_ino, unsigned long st_mode, unsigned long st_nlink, long st_uid, long st_gid, unsigned long st_rdev, array[2] of long st_pad2, long st_size, long st_pad3, struct timespec {..} st_atim, struct timespec {..} st_mtim, struct timespec {..} st_ctim, long st_blksize, long st_blocks, array[16] of char st_fstype, array[8] of long st_pad4}, int) returning int

sur mon programme c, j'effectue l'appel suivant :

Code :

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

ftw(nomRepertoireStats, traiteFS, 10);

Pouvez-vous m'aider ?
N'hésitez pas à me demander plus d'infos si besoin.

**Médinoc** · 08/10/2009, 14h48

Apparemment, tu as une fonction avec le prototype

Code C :

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

int NomFonction(char *, struct stat *, int);

Que tu passes en troisième paramètre d'une fonction demandant un pointeur sur fonction.

Tu dois modifier le prototype de ta fonction pour qu'il devienne ceci, et ça devrait compiler:

Code C :

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

int NomFonction(char const *, struct stat const *, int);

PS: Par curiosité, quel est ton compilateur?

**slm59** · 08/10/2009, 16h06

Merci pour ce retour rapide.

En effet, il s'agissait de la fonction traiteFS que j'ai donc modifiée :
int traiteFS(char const * entreeRep, struct stat const * statFic, int type)

Je n'ai plus l'erreur précédente mais de nouvelles erreurs liées certainement à cette modification.

"statsMens.c", line 28: warning: argument #1 is incompatible with prototype:
prototype: pointer to void : "/usr/include/iso/stdlib_iso.h", line 125
argument : pointer to const char
"statsMens.c", line 266: warning: argument #1 is incompatible with prototype:
prototype: pointer to char : "statsMens.c", line 244
argument : pointer to const char
"statsMens.c", line 271: warning: argument #1 is incompatible with prototype:
prototype: pointer to char : "statsMens.c", line 114
argument : pointer to const char
"statsMens.c", line 272: warning: argument #1 is incompatible with prototype:
prototype: pointer to char : "statsMens.c", line 244
argument : pointer to const char
"statsMens.c", line 290: left operand must be modifiable lvalue: op "="
cc: acomp failed for statsMens.c

Pour les erreurs sur les lignes 266,270 et 271, j'ai corrigé en remplaçant les arguments des autres fonctions pour que l'on pointe sur un "const char" et non un char.

Pour les lignes ci-dessous, je ne vois pas :
Ligne 28 : free(nomSgiv); cette commande est appelée dans la fonction void sortie(int code)

Ligne 290 : nomSgiv[0] = '\0';

D'avance merci pour votre aide.

PS : compilation avec SunStudio V11

**Médinoc** · 08/10/2009, 16h09

On ne peut pas faire un free sur quelque chose de const, donc il faudrait que j'en sache plus sur la fonction en question.
Pareil pour 290, où l'on tente apparemment de tronquer une chaîne const. Soit il y a une faille dans le design, soit je ne sais pas trop...

Toujours est-il que ni la fonction passée en paramètre, ni les fonctions qu'elle appelle, n'ont désormais le droit de modifier la chaîne ou les données contenues dans la structure pointée.

**slm59** · 08/10/2009, 16h22

Est-il possible de modifier le header ftw.h plutôt que de modifier le programme ?
C'est-à-dire de remplacer

Code :

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

extern int ftw(const char *, int (*)(const char *, const struct stat *, int),int);

par

Code :

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

extern int ftw(const char *, int (*)(char *, struct stat *, int),int);

Y'a-t-il un risque ?

**Médinoc** · 08/10/2009, 16h41

Je dirais qu'il y a toujours un risque à virer des const.
Faire ça au niveau du header, c'est encore pire que la solution "sale" que j'envisageais.

**slm59** · 08/10/2009, 16h54

Oups !! Je préfère passer pour un imbécile que faire une connerie

Quelle est la solution "sale" ?

**Médinoc** · 08/10/2009, 17h01

Mettre un cast "à la porc" dans la(les) fonction(s) qui a(ont) besoin de modifier la chaîne.

Mais je le répète, modifier/détruire la chaîne dans le callback est-il vraiment nécessaire?
Ne peux-tu absolument pas faire autrement? Car ce n'est, de toute évidence, pas la façon dont ftw() est supposée être utilisée.

**slm59** · 09/10/2009, 09h12

Bonjour,

J'ai retiré l'appel free qui semble-t-il ne sert pas (sauf cas extrême).

Il me reste donc le problème sur la ligne nomSgiv[0] = '\0';
La variable nomSgiv est déclaré précédemment de cette façon :
nomSgiv = (char *)malloc(LG_REP * sizeof(char));

Comment fonctionne le cast ?

**nicolas.sitbon** · 09/10/2009, 09h56

Il n'y a pas à discuter ici, le prototype de la fonction ftw() est

Code :

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

1
2
3
int ftw(const char *path, 
        int (*fn)(const char *, const struct stat *ptr, int flag), 
        int ndirs);

le pointeur de fonction passé en paramètre doit respecté le prototype, point barre.

Pour la fonction malloc(), la forme canonique est la suivante:

Code :

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

type identifiant = malloc(nombre_element * sizeof *identifiant);

ce qui donne dans ton cas:

Code :

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

char * nomSgiv = malloc(LG_REP * sizeof *nomSgiv);

Cordialement.

**Médinoc** · 09/10/2009, 09h59

Peux-tu poster un code minimal et compilable (plus ou moins) qui reproduit la situation?

Je voudrais comprendre exactement pourquoi l'on cherche à tronquer la chaîne dans la fonction callback plutôt qu'ailleurs...

**nicolas.sitbon** · 09/10/2009, 10h08

Envoyé par Médinoc

Je voudrais comprendre exactement pourquoi l'on cherche à tronquer la chaîne dans la fonction callback plutôt qu'ailleurs...

On ne peut pas la tronquer, la fonction ftw() n'est pas réentrante, la chaine dont tu parles est statique et représente successivement un objet dans le répertoire path.

**Médinoc** · 09/10/2009, 10h28

Ce n'est pas à moi qu'il faut dire ça.

**slm59** · 09/10/2009, 10h53

Bon, j'ai modifié la déclaration de la variable nomSgiv et à la compilation, je n'ai plus d'erreur.
Mais le binaire génère un core à l'exécution !

Je pense que c'est lié à l'allocation mémoire pour la variable nomSgiv qui n'est pas libérée.

Comment la libérer ?

**Médinoc** · 09/10/2009, 10h57

Poste ton code, s'il te plait.
Il nous faut tout depuis la déclaration des variables jusqu'à la fin de leur durée de vie, en passant par l'appel à ftw(), la fonction passée en callback et toutes les fonctions qu'elle appelle et qui travaillent sur la chaîne.

**Médinoc** · 09/10/2009, 11h39

C'est un nombre affreux de variables globales, ça.
Enfin si j'en crois la doc de ftw(), on n'a pas trop le choix.

J'examine le code d'un peu plus près...

**nicolas.sitbon** · 09/10/2009, 11h45

Envoyé par Médinoc

C'est un nombre affreux de variables globales, ça.
Enfin si j'en crois la doc de ftw(), on n'a pas trop le choix.

Je suis d'accord avec toi, mais les variables devraient au moins être déclarés statiques. Le code est trop horrible pour que je me penche dessus, ça ne tiendrais qu'à moi je referai tout, je n'en ai ni le temps ni l'envie, bon courage.

**Médinoc** · 09/10/2009, 11h54

Bon, j'ai réussi à faire compiler le code en C (pas pseudo-C++) sous mon visual sans ftw().

Je n'avais pas pigé au début qu'on bossait sur des globales. Je vais voir si je peux comprendre ce code avec les globales préfixées de g_...

**Médinoc** · 09/10/2009, 12h08

Je déclare officiellement ce thread mon liveblog du code de slm59.

Code C :

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

1
2
3
4
5
6
7
8
9
10
char * getNom(char const * path)
{
	/* extrait le base name du path passe en parametre */
	char * tmp = strdup(path);
	char * res = tmp;
 
	while (*tmp != '\0')
		if(*tmp++ == '/') res=tmp;
	return res;
}

Très joli. On cumule une réinvention de strrchr() et une fuite de mémoire.

Aussi, je me demande pourquoi on se fatigue à allouer dynamiquement les buffers en variable globale; je vais les allouer statiquement à la place.