Problème avec utilisation de execv

**Mr.Floppy** · 18/02/2016, 20h37

Bonjour à tous,

Voilà mon problème:

J'ai un programme très simple qui fonctionne très bien :

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
 
int main(int argc, const char * argv[])
{
    int choix;
 
    while (1)
    {
        printf("\nEntrez votre choix\n");
        scanf("%d",&choix);
        switch (choix)
        {
            case 1:
 
                printf("\nCas 1\n\n");
                break;
 
            case 2:
 
                printf("\nCas2\n\n");
                break;
 
            default:
                break;
        }
    }
}

Mon problème est que dès que je l'execute par le biais d'un execv, la console m'affiche "entrez votre choix " en boucle à l'infini....

Par avance merci!

**sambia39** · 18/02/2016, 22h05

Bonsoir
Ce n'est pas parce que votre programme fonctionne en saisissant des chiffres que le programme est correct.
Penché vous en détaille sur votre code vous remarquerez des erreurs.
Pour commencez, je vois une boucle infinie en clair votre programme ne s'arrête jamais deuxième point si l'on fait une erreur de saisie (admettant" que l'on tape "&" au lieu de 1 ) c'est la panique le programme s'affole.
Sécurisé votre saisie avant pour éviter des surprises.
Quant à l'erreur utilisant les primitives comment avez-vous implémenté le code.
Si possible pourriez-vous nous fournir le code ?

à bientôt

**Mr.Floppy** · 18/02/2016, 22h40

Bonsoir,

Merci pour votre aide, j'ai modifié mon code de manière à empêcher le programme de planter lors d'une saisie erronée :

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
 
int main(int argc, const char * argv[])
{
    char choix;
 
    while (1)
    {
        printf("\nEntrez votre choix\n");
        scanf("%s",&choix);
        switch (choix)
        {
            case '1':
 
                printf("\nCas 1\n\n");
                break;
 
            case '2':
 
                printf("\nCas2\n\n");
                break;
 
            default:
                break;
        }
    }
}

Malheureusement, ca n'a pas résolu mon problème, en fait mon code final ne ressemble pas à ca , je l'ai simplifié au maximum pour essayer de cibler l'erreur. C'est pour cela que j'en suis arrivé à un code aussi simple ( et qui ne sert à rien en l'état d'ailleurs!!

).
Ce que je voudrais c'est exectuer ce programme par le biais d'un execv dans un autre programme dont voici le code :

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
 
int main(int argc, char* argv[])
{
    int Planete1;
 
    char *chaine1[5]= {"/Users/christophe/projets/CellWars/testFork/prog", "5","5", NULL};
 
    Planete1=fork();
    if(Planete1==0)
    {
            sleep(1);
            execv("/Users/christophe/projets/CellWars/testFork/prog",chaine1);     // programme à executer
    }
 
    wait(&Planete1);
    exit(0);
    }

**Matt_Houston** · 19/02/2016, 10h11

Ce n'est pas le même programme puisque celui présenté ne traite pas ses arguments. Quel est le programme initial ?

**Mr.Floppy** · 19/02/2016, 10h55

Envoyé par Matt_Houston

Ce n'est pas le même programme puisque celui présenté ne traite pas ses arguments. Quel est le programme initial ?

Les 2 programmes sont tels que présentés, le premier programme fonctionne lorsqu'il est appelé seul mais pas lorsqu'il est appelé par le second(par le execv).

J'ai essayé de supprimer les chiffres passés en paramètres, et le problème reste le même.

**Médinoc** · 19/02/2016, 12h04

Envoyé par Mr.Floppy

Les 2 programmes sont tels que présentés, le premier programme fonctionne lorsqu'il est appelé seul mais pas lorsqu'il est appelé par le second(par le execv).

J'ai essayé de supprimer les chiffres passés en paramètres, et le problème reste le même.

Et si pour commencer, tu prenais en compte la valeur retournée par scanf()?
L'afficher quand elle est différente de 1 serait un bon début.
Quitter la boucle quand elle est négative serait une bonne suite.

Et même ça ne suffit pas: Il faut finir de lire la ligne, car si scanf("%d") tombe sur un caractère non-numérique, il ne le consommera pas et le caractère sera encore là pour le prochain appel de scanf("%d"), puis le suivant...

**sambia39** · 19/02/2016, 11h14

Bonjour

Envoyé par Matt_Houston

Ce n'est pas le même programme puisque celui présenté ne traite pas ses arguments. Quel est le programme initial ?

Effectivement, de plus il n'y a eu aucune modification, et si je peux me le permettre pourquoi utilisé des primitives ou un "fork" pour un programme qui ne nécessite pas l'utilisation de ces différente routine. Qu'est-ce qui justifie ce choix ? (sans oublier que l'on n'a pas vraiment le code d'origine qui génère l'anomalie en question.)
En attendant j'ai pris la peine d'apporter quelques modifications pour affirmer ce que j'ai énoncé dans mon premier poste mais attention, les codes source que je poste peut comporter des erreurs.

Le code source du choix sans execv et fork

Code C :

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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
 
/*
*  File: source.c
*  Version: 0.0.1 GPL V3.
*  Created by SAMBIA39 on 19/02/2016 - 9:34:46.
*  Copyright (c) 2016 SAMBIA39 and DEVELOPPEZ.NET
*/ 
#include <stdio.h>
#include <stdlib.h>
 
//type booléen
typedef enum e_bool{
    false,
    true
}bool;
 
/*
 *  Fonction qui joue similaire à perror 
 */
void f_fatal_err( char *p_msg ){
    extern int errno;
    extern const int sys_nerr;
    extern const char *const sys_errlist[];
 
    fprintf( stderr, "Erreur N°(%d)\t\t:%s\n", errno, p_msg );
    if( 0 < errno && sys_nerr > errno )
        fprintf( stderr, "Description erreur\t:%s\n", sys_errlist[errno] );
    else
        fprintf( stderr, "Description erreur\t:Erreur inconnue\n" );
 
    exit( EXIT_FAILURE );
}
 
/*  
*   Fonction qui se charge de prendre la saisie 
*   et vider le buffer elle renvoie vrais si 
*   la saisie a été correcte faut en cas d'erreur
*/
bool f_get_choix( char *ptr, size_t buff ){
 
    char *p_str = NULL;
    register int c_str;
    extern char *strchr();
    buff = ( 0 >= buff ) ? BUFSIZ : buff;
 
    fgets( ptr, buff, stdin );
    if( NULL != (p_str = strchr( ptr, '\n' ) ) )
        *p_str = '\0';
    else{
        do{
            c_str = getchar();
        }while( '\n' != c_str && EOF != c_str );
        return false;
    }
    return true;
}
 
/*
 *  Fonction principale
 */
int main( void ){
 
    int choix;
    char ptr[BUFSIZ];
    short i_bool = 1;
    extern int errno;
 
    while ( i_bool )
    {
        printf("\nEntrez votre choix\n");
        if( false == f_get_choix( ptr, BUFSIZ ) )
            f_fatal_err( "Saisie incorrecte" );
        sscanf(ptr, "%d", &choix);
        switch (choix)
        {
            case 1:
 
                printf("\nCas 1\n\n");
                break;
 
            case 2:
 
                printf("\nCas2\n\n");
                break;
            case 3:
                i_bool = 0;
                break;
            default:
                fprintf( stdout, "Commande invalide\n" );
                break;
        }
    }
 
    fprintf( stdout, "Fin du programme\n" );
    return EXIT_SUCCESS;
}

Le code source du avec fork et execv

Code C :

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
57
58
 
/*
*  File: source.c
*  Version: 0.0.1 GPL V3.
*  Created by SAMBIA39 on 19/02/2016 - 10:15:46.
*  Copyright (c) 2016 SAMBIA39 and DEVELOPPEZ.NET
*/ 
 
#include <stdio.h>
#include <stdlib.h>
 
/*
 *  Fonction qui joue similaire à perror 
 */
void f_fatal_err( char *p_msg ){
    extern int errno;
    extern const int sys_nerr;
    extern const char *const sys_errlist[];
 
    fprintf( stderr, "Erreur N°(%d)\t\t:%s\n", errno, p_msg );
    if( 0 < errno && sys_nerr > errno )
        fprintf( stderr, "Description erreur\t:%s\n", sys_errlist[errno] );
    else
        fprintf( stderr, "Description erreur\t:Erreur inconnue\n" );
 
    exit( EXIT_FAILURE );
}
 
/*
 *  Fonction principale
 */
int main(int argc, char *argv[]) {
 
    pid_t i_wait;
    pid_t i_fork;
 
    extern int errno;
    extern int wait();
    extern int execv();
    extern pid_t fork();
 
    errno = 0;
    switch ( i_fork = fork() ){
        case -1:
            f_fatal_err("Erreur fork");
	break;
	case 0:
            if( -1 == execv( "dev_test", argv) )
		f_fatal_err("Execution du programme");
            break;
	default:
            if( -1 == wait( &i_wait) )
		f_fatal_err("Erreur fork");
            break;
    }
 
    return EXIT_SUCCESS;
}

Le code source execs

Code C :

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
 
/*
*  File: source.c
*  Version: 0.0.1 GPL V3.
*  Created by SAMBIA39 on 19/02/2016 - 10:15:46.
*  Copyright (c) 2016 SAMBIA39 and DEVELOPPEZ.NET
*/ 
 
#include <stdio.h>
#include <stdlib.h>
 
/*
 *  Fonction qui joue similaire à perror 
 */
void f_fatal_err( char *p_msg ){
    extern int errno;
    extern const int sys_nerr;
    extern const char *const sys_errlist[];
 
    fprintf( stderr, "Erreur N°(%d)\t\t:%s\n", errno, p_msg );
    if( 0 < errno && sys_nerr > errno )
        fprintf( stderr, "Description erreur\t:%s\n", sys_errlist[errno] );
    else
        fprintf( stderr, "Description erreur\t:Erreur inconnue\n" );
 
    exit( EXIT_FAILURE );
}
 
/*
 *  Fonction principale
 */
int main(int argc, char *argv[]) {
 
 
    extern int errno;
    extern int execv();
 
    errno = 0;
    if( -1 == execv( "dev_test", argv) )
	f_fatal_err("Execution du programme");
    fprintf( stdout, "Fin de programme\n" );
 
    return EXIT_SUCCESS;
}