Passage d'un tableau à execvp() à partir de l'entrée de l'utilisateur

Votre problème, c'est que temp est un seul pointeur, et vous avez besoin pour passer un tableau de pointeurs sur des execvp().

Quelque chose comme:

    enum { MAX_ARGS = 64 };
    char *args[MAX_ARGS];
    char **next = args;

    temp = strtok(line, " ");
    while (temp != NULL)
    {
        *next++ = temp;
        printf("%s\n", temp);
        temp = strtok(NULL, " ");
    }
    *next = NULL;
    execvp(args[0], args);

Noter que la liste d'argument a été donné un pointeur null comme un terminator, tout comme argv[argc] == NULL dans main(). Clairement, j'ai lésiné sur la vérification d'erreur (si vous passez plus de 63 arguments, vous allez débordement args tableau). Mais il contient l'idée de base.

---

Avec cet exemple, je n'arrive pas à obtenir la commande simple de ls de travail, j'ai essayé mkdir et echo et ils semblent bien fonctionner. En passant ls retourne -1 à partir de execvp().

Je ne suis pas sûr de ce que peut être le problème — l'ensemble de ces travaux pour moi:

• ls
• ls -l
• ls -l madump.c (où madump.c arrive à être un fichier dans le répertoire que je suis en essais)

Le code que j'ai utilisé était:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>

int main(void) 
{
    char line[1024];

    while (fgets(line, sizeof(line), stdin) != NULL)
    {
        if (strcmp(line, "exit\n") == 0)
            exit(EXIT_SUCCESS);

        char *args[64];
        char **next = args;
        char *temp = strtok(line, " \n");
        while (temp != NULL)
        {
            *next++ = temp;
            printf("%s\n", temp);
            temp = strtok(NULL, " \n");
        }
        *next = NULL;

        puts("Checking:");
        for (next = args; *next != 0; next++)
            puts(*next);

        execvp(args[0], args);
    }

    return EXIT_SUCCESS;
}

Noter que j'ai ajouté \n à la strtok() liste des jetons, après la création d'un répertoire avec un saut de ligne à la fin de son nom. Bon pour les amis ennuyeux et incompréhensible semi-éduqués ennemis, mais une nuisance de la plupart des autres points de vue. Notez comment j'ai imprimer les données qui va être transmis à execvp() juste avant de le faire. Souvent, je ne l'utiliserais printf("<<%s>>\n", *next); au lieu de simplement puts() pour obtenir une claire indication de l'endroit où les arguments de début et de fin.

La sortie de l'exécution de la commande (doit) était:

$ ./doit
ls -l madump.c
ls
-l
madump.c
Checking:
ls
-l
madump.c
-rw-r--r--  1 jleffler  staff  2352 Jul 28  2011 madump.c
$

Qu'avez vous obtenez à partir de votre version?

OriginalL'auteur Jonathan Leffler

J'imagine que l'erreur est parce que vous êtes de passage à un pointeur NULL pour execvp. La boucle immédiatement au-dessus de l'appel garantit.

De faire ce que vous voulez faire, vous aurez besoin de créer un tableau de pointeurs de chaîne pour ce que vous avez actuellement appelé temp. Cela devient le argv tableau pour le programme que vous appelez. C'est pourquoi la commande est généralement utilisée comme execvp (temp[0], temp) - argv[0] est généralement le nom du programme.

Donc, essayez de créer un tableau de pointeurs de chaîne, et ont chacun un point à un sous word à partir de line. Vous devrez peut-être utiliser malloc, mais si vous voulez faire le malin, vous pourriez être en mesure de pointer directement dans line. Si vous faites cela, vous aurez besoin de définir le caractère immédiatement après chaque " mot " à \0.

OriginalL'auteur lxop