Question

我目前正在创建自己的命令行shell，并且在尝试接收管道时遇到问题。我的程序从父进程开始。它检查用户是否放入退出或历史记录。如果是这样，它会使用这些命令，但这并不重要。如果用户输入除退出或历史之外的任何内容，则会创建执行当前任务的子项。

但是，如果用户输入一个具有单个管道的命令，那么我们从父进程开始，它创建一个子进程，调用此子进程，子进程1.子进程1看到有一个管道并使用fork来创建另一个孩子，调用这个孩子2.孩子1创建另一个孩子，称之为孩子3（注意：孩子2和3使用共享内存）。现在，子2执行第一个命令，子3使用子2的输出执行命令。但由于某种原因，我没有得到任何输出。

我不确定这是否是执行任务的最有效方式，但这是我教授所说的。

如果您想在此处查看我的所有代码，请执行以下操作：http://pastebin.com/YNTVf3XP

否则这是我从代码1开始的代码：

            //Otherwise if we have a single pipe, then do
            else if (numPipes == 1) {

                char *command1[CMD_MAX];        //string holding the first command
                char *command2[CMD_MAX];        //string holding the second command
                int fds[2];                     //create file descriptors for the parent and child
                pid_t new_pid;                  //create new pid_t obj

                onePipeSplit(tokens, command1, command2, numTokens);    //Getting each command for pipe

                if (pipe(fds) < 0) {            //use the pipe command. If < 0
                    perror("Pipe failure.");    //we have an error, so exit
                    exit(EXIT_FAILURE);
                }

                //Creating child 2
                new_pid = fork();

                //if pid < 0 then we have an error. Exit.
                if (new_pid < 0) {
                    perror("Fork failure.");
                    exit(EXIT_FAILURE);
                }

                //Else we have the child (2) process
                else if (new_pid == 0) {

                    close(fds[0]);      //Close read

                    //sending stdin to the shared file descriptor
                    if (dup2(fds[1], STDOUT_FILENO)<0) {
                        perror("Can't dup");
                        exit(EXIT_FAILURE);
                    }

                    execvp(command1[0], command1);  //Execute the next command
                    perror("Exec failure");
                    exit(EXIT_FAILURE);
                }

                //Else if we have the parent process
                else {

                    pid_t child = fork();           //Creating child 3
                    if (new_pid < 0) {              //if pid < 0, error, exit
                        perror("Fork failure.");
                        exit(EXIT_FAILURE);
                    }

                    //else if pid > 0 we have the parent wait until children execute
                    else if (new_pid > 0) {
                        wait(NULL);
                    }

                    //else we have the child (3)
                    else {

                        close(fds[1]);      //Close write

                        //Sending stdout to the shared file descriptor
                        if (dup2(fds[0], STDIN_FILENO) < 0) {
                            perror("Can't dup");
                            exit(EXIT_FAILURE);
                        }

                        execvp(command2[0], command2);  //Execute the command
                        perror("Exec failure");
                        exit(EXIT_FAILURE);
                    }
                }
            }

这是我教授给我的一张图片，展示了它应该如何运作。

Answer 1

问题在于：

                   pid_t child = fork();           //Creating child 3
                    if (new_pid < 0) {   
       ... you keep checking `new_pid` here on down,
       ... but you should be checking `child` here on down...

同样在onePipeSplit中，您需要在两个命令列表的末尾放置一个NULL，因为execvp需要它。在第一个循环之后添加：

command1[i] = NULL;

并在第二次之后：

command2[i] = NULL;

好的，还有一些修复：

每个dup2（）之后你需要关闭原来的fd。一个例子：

                    if (dup2(fds[1], STDOUT_FILENO)<0) {
                        perror("Can't dup");
                        exit(EXIT_FAILURE);
                    }
                    close(fds[1]);      /* ADD ME */

并在父进程中：

                    //else if pid > 0 we have the parent wait until children execute
                    else if (child > 0) {
                        close(fds[0]);   /* we don't use */
                        close(fds[1]);   /* the pipe */
                        wait(NULL);
                        exit(0);         /* when 2 & 3 are done, we are too */
                    }

为什么不打印到标准输出（标准输出）？

1 个答案: