Ошибка при использовании каналов и команды exec.Second не завершается

Question

Код принимает команду в качестве ввода и выполняет ее. Трубы также обрабатываются. Проблема в том, что если я введу ls | grep x как команда. Процесс grep не завершается, и программа останавливается. Есть идеи.

#include <stdio.h>
#include <unistd.h>
#include <string.h>
#include <signal.h>
void parse(int *cont,char *command,char **exarg)
{
    char *x=strchr(command,'&');
    if(x!=NULL)
    {
        *cont=1;
        *(--x)='\0';
    }
    x=strtok(command," ");
    while(x!=NULL)
    {
        *exarg++=x;
        x=strtok(NULL," ");
    }
    *exarg='\0';
}

int divide(char *cmd,char **cmdarr)
{
    int i=0;
    char *x;
    x=strtok(cmd,"|");
    while(x!=NULL)
    {
        i++;
        *cmdarr++=x;
        x=strtok(NULL,"|");
    }
    *cmdarr='\0';
    return i;
}

void main()
{
    pid_t pid;
    int cont;
    int i=0;
    int orgnum;
    char *cmdarr[50];
    char c[50];
    int p1[2];
    int p2[2];
    char *argument[50];
    if(pipe(p1)<0)
        printf("error\n");
    if(pipe(p2)<0)
        printf("error2\n");
    for(;;)
    {
        printf("Enter the command:");
        gets(c);
        if(!strcmp("exit",c))
            break;
        orgnum=divide(c,cmdarr);
        printf("%d\n",orgnum);
        for(i=0;i<orgnum;i++)
        {
            pid=fork();
            if(pid==0)
            {
                if(i==0)
                {
                    close(p1[0]);
                    close(p1[1]);
                    close(p2[0]);
                    if((i+1)!=orgnum)
                    {
                        dup2(p2[1],STDOUT_FILENO);
                        dup2(p2[1],STDERR_FILENO);
                    }
                    close(p2[1]);
                    parse(&cont,cmdarr[i],argument);
                    if((execvp(argument[0],argument))<0)
                    {
                        printf("Wrong cmd");
                        exit(0);
                    }
                }
                else if(i>0)
                {
                    if((i+1)%2==0)
                    {
                        close(p1[0]);
                        close(p2[1]);
                        dup2(p2[0],STDIN_FILENO);
                        close(p2[0]);
                        if((i+1)!=orgnum)
                        {
                            dup2(p1[1],STDOUT_FILENO);
                            dup2(p1[1],STDERR_FILENO);
                        }
                        close(p1[1]);
                    }
                    else if((i+1)%2==1)
                    {
                        close(p2[0]);
                        close(p1[1]);
                        dup2(p1[0],STDIN_FILENO);
                        close(p1[0]);
                        if((i+1)!=orgnum)
                        {
                            dup2(p2[1],STDOUT_FILENO);
                            dup2(p2[1],STDERR_FILENO);
                        }
                        close(p2[1]);
                    }
                    parse(&cont,cmdarr[i],argument);
                    if((execvp(argument[0],argument))<0)
                    {
                        printf("Wrng cmd");
                        exit(0);
                    }
                }
            }
            else if(pid>0){
                close(p1[0]);
                close(p1[1]);
                close(p2[0]);
                close(p2[1]);
                wait(NULL);}
        }
    }
}

Answer 1

Итак, у вас есть цикл:

вилка, если ребенок
Exec
иначе, если родитель
закрыть трубы
Подождите

Когда у вас есть что-то вроде "ls | grep foo",

шаг 1: вы исполняете ls в child, закрываете все каналы и ждете завершения 'ls' в parent,

шаг 2: exec 'grep' в child и использовать pipe в качестве входных данных, но вы закрываете их в шаге 1.