Discussion :

[slm59]

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]

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]

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]

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]

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]

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.