程序奇怪地回到循环的顶部

Question

我想在这个问题前面说我是C的新手并因此很糟糕,所以我提前为任何明显的错误或糟糕的风格道歉.此外,我不知道如何在向您展示我的代码之前介绍问题,所以这里是:

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


int main()
{

    int MAX_INPUT_SIZE = 200;
    volatile int running = 1;
    while (running)
    {

        char input[MAX_INPUT_SIZE];
        char *tokens[100];
        const char *cmds[] = { "wait", "pwd", "cd", "exit" };
        char *cmdargs[100];

        printf("shell> ");
        fgets(input, MAX_INPUT_SIZE, stdin);

        // remove newline character at end
        int nl = strlen(input) - 1;
        if (input[nl] == '\n')
        {
            input[nl] = '\0';
        }

        // tokenize input string, put each token into an array
        char *space;
        space = strtok(input, " ");
        tokens[0] = space;

        int i = 1;
        while (space != NULL)
        {
            space = strtok(NULL, " ");
            tokens[i] = space;
            ++i;
        }

        // copy tokens after first one into string
        int noargscheck;
        if (tokens[1] != NULL)
        {
            noargscheck = 0;
            strcpy((char *)cmdargs, tokens[1]);
            for (i = 2; tokens[i] != NULL; i++)
            {
                strcat((char *)cmdargs, " ");
                strcat((char *)cmdargs, tokens[i]);
            }
        }
        else
        {
            noargscheck = 1;
        }

        // compare tokens[0] to list of internal commands
        int isInternal = -1;
        for (i = 0; i < 4; i++)
        {
            if (strcmp(tokens[0], cmds[i]) == 0)
            {
                isInternal = i;
            }
        }


        // internal commands
        char wd[200];
        if (isInternal != -1)
        {
            switch (isInternal)
            {
            case 0:
                // wait
                break;
            case 1:
                // pwd
                if (getcwd(wd, sizeof(wd)) == NULL)
                {
                    perror("getcwd() error!");
                }
                else
                {
                    printf("%s\n", wd);
                }
                break;
            case 2:
                // cd
                if (noargscheck)
                {
                    chdir("/home");
                }
                else if (chdir((const char *)cmdargs) != 0)
                {
                    perror("cd failed");
                }
                break;
            case 3:
                // exit
                exit(1);
                break;
            }
        }

        else
        {
            // external commands

            pid_t child_pid;
            switch (child_pid = fork())
            {
            case -1:
                perror("Fork failed");
                return 1;
            case 0:
                // child
                printf("\nHERE\n"); // for debugging
                execvp(tokens[0], cmdargs);
                break;
            }
        }
    }
}

当我使用输入运行此代码时echo hello world,程序成功地case 0在第二个switch语句中输入大小写,switch (child_pid=fork())但输出是意外的,如下所示:

输出:(包括在提示符下显示我的输入的一行)

shell> echo hello world (我的意见)

shell> (这是我不明白的部分)

HERE

shell> (程序现在等待下一个用户输入的提示)

我无法弄清楚为什么shell>要打印额外的提示.任何人都能看到问题吗？

编辑:修复了execvp的第一个参数.改变了"echo"(那是因为我很傻)tokens[0].

Answer 1

当你分叉时,你现在有两个进程.您的孩子将打印该HERE消息,然后致电execvp.您不检查返回值,execvp因此可能返回错误.cmdargs应该是一个向量 - 即一个以空指针终止的字符串指针数组.相反,你传递execvp一个字符串.换句话说,它需要一个char* [],这是什么cmdargs,然而你以前处理cmdargs不正确.

例如,你说strcpy((char*)cmdargs, tokens[1]);.这会放一个字符串*cmdargs.字符串是一个零个或多个非零字符的数组,后跟一个NUL8位宽的ascii :

char* cmdargs[] is a double pointer
you treat cmdargs as a single pointer and feed it to strcpy
cmdargs points to: 'H' 'e' 'l' 'l' 'o' '\0'

但是,这不是execvp想要的.Execvp需要一个向量,看起来有点像这样:

char* cmdargs[] is a double pointer
cmdargs[0] is a single pointer
cmdargs[0] points to: 'H' 'e' 'l' 'l' 'o' '\0'
cmdargs[1] points to: 'w' 'o' 'r' 'l' 'd' '\0'
cmdargs[2] is a null pointer, indicating that it is the end of the vector

因此,execvp无法找到向量的结尾并失败,返回-1.您没有对此进行测试,因此子进程继续回到循环的顶部,父进程也是如此,并且两个进程都打印出来shell>.

编辑:顺便说一下,argv向量中的FIRST字符串应该是正在执行的文件的名称 - 在这种情况下,echo然后第二个和第三个字符串应该是第一个和第二个"参数" - 这里hello和world.这是输入到您调用的程序的argv向量,按照惯例,该向量中的第一个元素是被调用程序的名称.如果你忽略那个惯例,echo会非常困惑.