Discussion :

[Lyysis]

bonjour tout le monde,
j'ai un petit problème avec la fonction execlp:

Mais commençons par le début: je génère un menu en fonction du contenue de ce qui se trouve dans mon répertoire donc, je stocke tous mes fichiers .c dans un tableau et mon utilisateur choisit.

Sauf que quand je fais mon execlp sur mon .c ça me fait une erreur de segmentation (core dumped) et je ne comprend pas pourquoi :/

un petit bout de code pour mieux cerner le problème:

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
 
 
	printf("Choice: \n");
	scanf("%d", &choice);
	printf("\n");
	for (i=0;i<cptBis;i++){
		printf("%s\t", tabChoix[i]);
	}
	printf("%d\t", cptBis);
	if(choice>='0' && choice<cptBis){
		printf("%s\n", tabChoix[choice]);
		if(execlp(tabChoix[choice], tabChoix[choice], PROC, nbProc, NULL)==-1){	
			puts("erreur1");
		}
 
	}else puts("erreur2");

[diogene]

Il y a très probablement une erreur ici due aux ' ':

Code :

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

if(choice>=0 && choice<cptBis){

[Lyysis]

Je ne pense pas que ça implique une erreur car choice est un char et il rentre quand même dans la boucle.

[diogene]

Cela fait une différence : dans le cas de '0', c'est équivalent à choice >= 48 (si tu es en ASCII) au lieu de choice positif ou nul.

[Lyysis]

Sur tes conseils (et comme dans la seconde partie je le compare à un int) je l'ai modifié mais ça ne résout malheureusement pas mon problème de base :s .

[plxpy]

Bonjour

je l'ai modifié mais ça ne résout malheureusement pas mon problème de base :s

Et bien, justement. Je ne comprends ta façon de faire. Si tu soupçonnes une mauvaise utilisation de ta part de execlp, pourquoi testes-tu du code où l'appel à execlp est véritablement noyé dans plein d'autres choses qui, apparemment - tes échanges avec diogene - ne sont pas "nickel" ?

Ecris un petit - minuscule - petit programme jetable (cible_execlp) comme celui ci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <stdio.h>
#include <stdlib.h>
int main(int argc,char *argv[0])
{
    int i;
 
    fprintf(stdout,"je suis %s\n",argv[0]);
    if (argc == 1)
        fprintf(stdout,"je n'ai pas recu d'argument\n");
    else
        for (i=1;i<argc;i++)
            fprintf(stdout,"argument %d : %s\n",i,argv[i]);
    return 0;
}

et un autre, jetable lui aussi, minimaliste, qui validera ce que tu as compris de l'utilisation de execlp (un "appel" de cible_execlp via execlp). J'en ai fait un (appelant_execlp.c) qui fait, royalement 10 lignes et tout ça donne :

Code :

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

1
2
3
4
5
6
7
plx@sony:~$ ./appelant_execlp 
je vais de suite appeler execlp sur "cible_execlp" avec les arguments "1" "deux" "3" NULL
je suis cible_execlp
argument 1 : 1
argument 2 : deux
argument 3 : 3
plx@sony:~$

Tu sauras, une bonne fois pour toutes, si tu as bien compris le fonctionnement de execlp et de ses arguments et, si ça marche, tu sauras que c'est lors de l'appel que ça "foire".