试图将用户输入传递到我正在创建的简单外壳中

Question

我正在创建一个简单的外壳。用户输入一个系统调用，对于用户输入的任何命令，我在子进程上分叉并调用 execvp。我进行了调试，似乎我传递给 execvp 的参数不正确。有人可以帮帮我吗，我是C的新手

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <unistd.h>
#define MAXLINE 80
int main(void){
    char *args[MAXLINE/2 + 1]; 
    char buf[MAXLINE/2 + 1];

    int should_run = 1; 
    printf("Shell\n");
    fgets(buf, MAXLINE,stdin);
    buf[strlen(buf)-1] = '\0';
    int i=0;
    char *token;
    token = strtok(buf," ");
    while(token != NULL){
        args[i++] = token;
        token = strtok(NULL," ");
    }

    while(should_run){
        if(strcmp(args[0],"exit") == 0){
            should_run = 0;
            break;
        }
        pid_t pid;
        pid = fork();

        //child process
        if(pid == 0){
            execvp(args[0],args);
        }
        else{

            wait(0);
            //Also how do I make it so that the program starts from top again, meaning it lets the user enter another system call
        }
    }
}

Answer 1

execvp 函数要求您以 NULL 终止列表本身。目前，您的标记化循环不会在最后一个标记上放置 NULL，因此行为未定义（您从未初始化过数组）。

一个紧凑（但不是很漂亮）的解决方案是像这样重新排列你的循环：

int i = 0;
char *token;
do {
    token = strtok( i == 0 ? buf : NULL," ");
    args[i++] = token;
} while( token != NULL );