为什么不能和execvp一起工作?

时间:2013-11-18 16:15:40

标签: c linux exec

我有一个作业,要求我编写一个迷你shell - 这将获得执行命令,执行它,并等待更多命令。

当我传递给这个迷你shell命令ls .时,它打印当前目录的比赛。当我传递给它ls时,它什么都不打印。为什么呢?

这是我的代码:

#include <stdio.h>
#include <sys/types.h>
#include <sys/wait.h>
#include <string.h>
#include <unistd.h>
#include <assert.h>
#include <stdlib.h>
#include <stdbool.h>
#define MAX_CMD_SIZE 40
char** parse(char*);//will parse the arguments for the execv/excevp commands.

int main(int argc, char** argv)
{
    bool debug = false;
    assert(argc <= 2);
    if (argc == 2)
    {
        //check for string -debug
        debug = true; 
    } 
    if (debug)
        printf("INFO: Father started PID[%d]\n", getpid());
    char *command = malloc(MAX_CMD_SIZE);
    while(true)
    {
        printf("minishell> ");
        fgets(command, MAX_CMD_SIZE, stdin);
        if (strcmp(command, "exit\n") == 0)
            return 0;
        pid_t pid = fork();
        assert(pid >= 0); 
        if (pid == 0) //child
        {
            if (debug)
                printf("INFO: Child started PID[%d]\n", getpid());
            char** buf = parse(command);
            if (debug)
            {
                int i;
                for (i = 0; buf[i]; i++)
                    printf("INFO: buf[%d] = %s\n",i,buf[i]);
            }
            execvp(buf[0],buf);
            return 0;
        }
        else //father
        {
            int status;
            wait(&status);
            if (debug)
                printf("INFO: Child with PID[%d]terminated, continue waiting commands\n", pid);
        }
    }
}

char** parse(char *string)
{
    char** ret = malloc(sizeof(char*));
    ret[0] = strtok(string, " ");
    int i = 0;
    for (; ret[i]; ret[i] = strtok(NULL, " \n"))
    {
        ret = realloc(ret, sizeof(char*) * ++i);
    }

    return ret;
}

1 个答案:

答案 0 :(得分:3)

您的parse()命令在最后一个参数中包含\n:)

所以使用单个ls,你实际上正在执行ls\n,它不在PATH中(当然)

问题是,在第一次strtok()调用时,您只能将" "作为分隔符传递。使用" \n"(就像在后续调用中一样),问题就会消失。

你也可以通过扼杀\n

来解决这个问题
int l = strlen (string);
if (l > 0 && string [l - 1] == '\n') string [l - 1] = '\0';

并且只使用" "作为分隔符。