1

我非常接近于弄清楚我一直在为用 C 编写的 linux shell 编写的程序。我一直想让它工作一段时间，我决定再次拿起它，并在过去一直在修补它几个星期。

对于以下代码，请记住名为arrayOfCommands的数组是动态填充的。我的代码用正在运行的当前命令填充 arrayOfCommands。为了我的示例，我们将运行命令ls -l | wc和 arrayOfCommands 填充以下内容，具体取决于循环的时间：

//Pass #1
arrayOfCommands[]= ("ls", "-l", NULL)

//Pass #2
arrayOfCommands[]= ("wc", NULL)

这是我到目前为止所拥有的：

//PIPING
int do_command(char **args, int pipes) {
    // pipes is the number of pipes in the command
    // (In our example, one)


    // The number of commands is one more than the
    // number of pipes (In our example, two)
    const int commands = pipes + 1;  //Ex: 2
    int i = 0;

    // Set up the pipes
    int pipefds[2*pipes];

    for(i = 0; i < pipes; i++){
        if(pipe(pipefds + i*2) < 0) {
            perror("Couldn't Pipe");
            exit(EXIT_FAILURE);
        }
    }

    // Variables
    int pid;
    int status;
    char *str_ptr;

    int j = 0;
    for (i = 0; i < commands; ++i) {

        // A magic function that updates arrayOfCommands with
        // the current command goes here.  It doesn't make 
        // sense in the context of the code, so just believe me! :)

        // Ex: The contents will be "ls -l" or "wc" depending on 
        // which time through the loop we are

        pid = fork();

        if(pid == 0) {
            //if not last command
            if(i < commands){
                if(dup2(pipefds[j + 1], 1) < 0){
                    perror("dup2");
                    exit(EXIT_FAILURE);
                }
            }

            //if not first command&& j!= 2*pipes
            if(j != 0 ){
                if(dup2(pipefds[j-2], 0) < 0){
                    perror("dup2");
                    exit(EXIT_FAILURE);
                }
            }

            for(i = 0; i < 2*pipes; i++){
                    close(pipefds[i]);
            }

            // Should any of the below inputs be *arrayOfCommands or 
            // **arrayOfCommands or &arrayOfCommands?
            // I'm REALLY bad with pointers

            if( execvp(arrayOfCommands, arrayOfCommands) < 0 ){
                    perror(arrayOfCommands);
                    exit(EXIT_FAILURE);
            }
        }
        else if(pid < 0){
            perror("error");
            exit(EXIT_FAILURE);
        }

        j+=2;
    }

    for(i = 0; i < 2 * pipes; i++){
        close(pipefds[i]);
    }

    for(i = 0; i < pipes + 1; i++){
    }
        wait(&status);
}

当我运行它时，我得到了几个错误：

  • dup2：错误的文件描述符
  • ls: |: 没有那个文件或目录
  • ls: wc: 没有这样的文件或目录

有人可以帮我弄清楚以下两件事：

  1. 为什么我会收到这些错误？
  2. 在 execvp 函数中，我在寻找什么样的指针？ arrayOfCommands 被初始化为 char *arrayOfArgs[]
4

1 回答 1

3

第一件事：

//if not last command
if(i < commands)

应该

if(i < commands -1)

因为i从0到那commands -1 应该解决 dup2: Bad file descriptor

ls: |: 没有那个文件或目录

ls: wc: 没有这样的文件或目录

都是畸形造成的 arrayOfCommands。它必须由

char * arrayOfCommands[] = {"ls", "-l", NULL};

和

char * arrayOfCommands[] = {"wc", NULL};

分别并通过调用execvp(arrayOfCommands[0], arrayOfCommands)

本质上必须与您的参数向量（通常）具有arrayOfCommands相同的格式。argvint main(int argc, char** argv)

于 2012-10-01T00:11:44.807 回答