如何链接多个管道? (加上怪异的bug)

时间:2016-11-19 18:09:47

标签: c shell pipe fork pipeline

我正在尝试编写一个可以处理管道命令的简单shell。我希望能够处理所有链接在一起的多个管道,但我很难弄清楚如何实现这样的事情。

这是我目前的尝试:

int status;
    int lastToken = 0;
    int pipe_pid;

    //create the pipes
    int pipefd[pipes][2];

    // Loop to run all commands in the vertical list.
    while(1){
        if (c->type == TOKEN_PIPE){

        // Here is where we deal with pipes
            for (int i = 0; i < pipes; i++){
                pipe(pipefd[i]);

                pipe_pid = fork();

                //this is a receiving pipe
                if (pipe_pid == 0){
                    // create the write end of the pipe
                    dup2(pipefd[i][WRITE_SIDE], STDOUT_FILENO);
                    close(pipefd[i][READ_SIDE]);
                    close(pipefd[i][WRITE_SIDE]);
                    execvp(c->argv[0], c->argv);
                    // printf("parent pipe\n");
                }
                //this is a writing pipe
                else{
                    close(pipefd[i][WRITE_SIDE]);
                    dup2(pipefd[i][READ_SIDE], STDIN_FILENO);
                    close(pipefd[i][READ_SIDE]);
                    // printf("child pipe\n");
                }
            }



        // This stuff happens for all commands
        lastToken = c->type;
        // If it's the last command, we're done
        if (c->next == NULL){
            break;
        }
        else{
            c = c->next;
        }
    }

命令在链表中链接在一起,c是我的命令指针

管道是我在解析字符串时创建的变量,所以我知道有多少'|'我在命令中看到了。这应该告诉我需要分叉的子进程数。

我使用管道为管道描述符创建一个二维数组。

然后我想循环管道和fork一次,并使用dup2映射输入和输出。

我得到了一些我无法弄清楚的错误。首先,每次运行管道命令时,我的shell都会立即崩溃,没有任何段错误或其他打印错误。

其次,如果我运行像echo foo | wc -c这样的命令,我有时会得到4,有时会得到0作为输出。

我确定我只是在做一些愚蠢的事情,但我不确定是什么:/

1 个答案:

答案 0 :(得分:0)

我弄清楚我做错了什么,我在所有线程完成使用之前关闭了管道。我通过拉出近距离通话来修复它。

// writing side of the pipe
if (c->type == TOKEN_PIPE){
    close(c->pipefd[READ_SIDE]);
    dup2(c->pipefd[WRITE_SIDE], STDOUT_FILENO);

}
// receiving side of the pipe
if (commandPrev->type == TOKEN_PIPE){
    close(commandPrev->pipefd[WRITE_SIDE]);
    dup2(commandPrev->pipefd[READ_SIDE], STDIN_FILENO);

}

然后在父线程中,在我恢复僵尸之前,我检查已完成使用的管道并关闭它们。

    // writing side of the pipe
    if (c->type == TOKEN_PIPE){
        close(c->pipefd[READ_SIDE]);
        dup2(c->pipefd[WRITE_SIDE], STDOUT_FILENO);
    }
    // receiving side of the pipe
    if (commandPrev->type == TOKEN_PIPE){
        close(commandPrev->pipefd[WRITE_SIDE]);
        dup2(commandPrev->pipefd[READ_SIDE], STDIN_FILENO);
    close(commandPrev->pipefd[READ_SIDE]);

我不确定这是否是最佳方式,但它对我来说没有错误。