Discussion :

[progdevlop]

Bonjour
J'ai écris un programme qui permet de créer un processus fils, d'afficher le PID et l'état de chaqun des deux processus (le père et le fils), seulement le processus fils va faire une tâche : il va appeler (ou executer) un autre programme que j'ai écris.
Le problème : après la compilation du premier programme j'ai eu les erreurs suivantes :

processu.c: In function ‘main’:
processu.c:29: warning: missing sentinel in function call

Si quelqu'un peut m'aider pour résoudre ce problème ?
Merci d'avance.

Voici le code source des deux programmes :

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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
 
 /*------code source de processus.c:-------*/
 
 #include<unistd.h>
 #include<stdio.h>
 #include<stdlib.h>
 int fils,pidfils,iden;
 char etat1[1024];
 char etat2[1024];
 main()
 {
 int pid_p;
 pid_p=getpid();
 printf("pere: je suis le processus pere mon pid=%d\n",getpid());
 sleep(1);
 sprintf(etat1,"ps -o s -p%d",pid_p);
 system(etat1);
 iden=fork();
 if(iden==-1)
 {
 printf("erreur de creation de processus fils\n");
 exit(1);
 }
 if(iden==0)
 {
 int pid_f;
 pid_f=getpid();
 printf("\nfils: je suis le processus fils mon pid=%d\n\n",pid_f);
 sprintf(etat2,"ps -o s -p%d",pid_f);
 system(etat2);
 sleep(1);
 execl("prog","prog",0);
 }
 else
 {
 printf("\npere: maintenant je dors 5 secondes\n");
 sleep(5);
 printf("pere: je suis reveille\n\n");
 pidfils=wait(&fils);
 printf("pere: mon processus fils termine son execution et son pidfils=%d\n\n",pidfils);
 }
 }
 
 /*-----------code source de prog.c-----------*/
 #include<sys/types.h>
 #include<sys/wait.h>
 
 main()
 {
 printf("\n\t*************PROGRAMME EXECUTE PAR LE FILS******************\n");
 printf("\nle programme: je suis le programme executer par le fils\n");
 printf("\nle programme: l'identificateur de mon pere ppid=%d\n",getppid());
 printf("\nle programme: le fils va vous donnez la date d'haujourd'huit\n");
 printf("\nle programme:\t");
 execlp("/usr/bin/date","/usr/bin/date",NULL);
 }

[Médinoc]

La ligne 29, ne serait-ce pas celle de l'appel à execl() ?

Si je peux me fier à cette doc, il est possible que le zéro doive être explicitement casté en char*.

[publicStaticVoidMain]

LE dernier paramètre de execl doit NULL.

[Obsidian]

J'ajoute, pour compléter la réponse de publicStaticVoidMain, qu'une sentinelle est une valeur balise que l'on place à la fin d'une suite de données, précisément pour détecter cette fin le moment venu sans que l'on ait à tenir un compteur à jour pour le savoir.

À noter également que execl est une fonction variadique et que le type des paramètres n'est pas défini par défaut. Le compilateur peut reconnaître le paramètre passé sur la forme et générer un avertissement mais, dans tous les cas, comme je n'ai jamais travaillé sur des plate-formes qui définissaient NULL autrement qu'avec un zéro, ça devrait fonctionner quand même malgé le warning. Ce n'est bien sûr pas une raison pour le laisser en l'état. :-)

[nicolas.sitbon]

comme je n'ai jamais travaillé sur des plate-formes qui définissaient NULL autrement qu'avec un zéro,

moi c'est le contraire, pour le moment, toutes les plateformes sur lesquelles j'ai bossé définissait NULL correctement (== qui fonctionne dans tous les cas) à savoir:

Code :

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

#define NULL (void*)0

[Médinoc]

Du moins, en C. Quand on passe au C++, c'est une autre histoire.

[Obsidian]

moi c'est le contraire, pour le moment, toutes les plateformes sur lesquelles j'ai bossé définissait NULL correctement (== qui fonctionne dans tous les cas) à savoir:

Code :

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

#define NULL (void*)0

C'est ce que je voulais dire. La valeur numérique est un zéro. Une fois compilée, c'est bien cette valeur que l'on retrouve chargée dans le code assembleur.

[progdevlop]

J'ajoute, pour compléter la réponse de publicStaticVoidMain, qu'une sentinelle est une valeur balise que l'on place à la fin d'une suite de données, précisément pour détecter cette fin le moment venu sans que l'on ait à tenir un compteur à jour pour le savoir.

À noter également que execl est une fonction variadique et que le type des paramètres n'est pas défini par défaut. Le compilateur peut reconnaître le paramètre passé sur la forme et générer un avertissement mais, dans tous les cas, comme je n'ai jamais travaillé sur des plate-formes qui définissaient NULL autrement qu'avec un zéro, ça devrait fonctionner quand même malgé le warning. Ce n'est bien sûr pas une raison pour le laisser en l'état. :-)

Justement il s'execute normalement malgé le warning mais ce même problème s'est affiché dans un autre programme que j'ai écris mais je n'est pas mis l'instruction execl() (l'execution du programme marche normalement), seulement je veux comprendre ces erreurs sont dû à quoi ?

[nicolas.sitbon]

Justement il s'execute normalement malgé le warning mais ce même problème s'est affiché dans un autre programme que j'ai écris mais je n'est pas mis l'instruction execl() (l'execution du programme marche normalement), seulement je veux comprendre ces erreurs sont dû à quoi ?

C'est fonction sont déclarés avec un attribut GCC particuliers nommé sentinel:

sentinel
This function attribute ensures that a parameter in a function call is an explicit NULL. The attribute is only valid on variadic functions. By default, the sentinel is located at position zero, the last parameter of the function call. If an optional integer position argument P is supplied to the attribute, the sentinel must be located at position P counting backwards from the end of the argument list.

__attribute__ ((sentinel))
is equivalent to
__attribute__ ((sentinel(0)))


The attribute is automatically set with a position of 0 for the built-in functions execl and execlp. The built-in function execle has the attribute set with a position of 1.

A valid NULL in this context is defined as zero with any pointer type. If your system defines the NULL macro with an integer type then you need to add an explicit cast. GCC replaces stddef.h with a copy that redefines NULL appropriately.

The warnings for missing or incorrect sentinels are enabled with -Wformat