c - 帮助我的 c shell 中的管道工作

Question

您好，我需要一些关于我为 C shell 创建的管道方法的帮助。我想让它运行一个简单的 ls |厕所。

我编写了一个名为 createPipe 的方法，它 fork 两个子进程，因为 cmd1 的标准输出成为 cmd2 的标准输入。

我还编写了一个名为 execute() 的方法，它应该拆分我们必须维护一个恒定路径变量的路径)并 execv 路径和命令，我必须使用 execv，而不是 execvp 或 execlp 等。

当我在 createPipe 的部分执行我的程序时，它命中 execute(command)，我一直得到 ls | wc 是无效命令。

如果我尝试仅execv createPipe 方法中的命令而不是使用execute 方法，它会遇到我插入的错误我的方法，所以我得到:

no exec child1: bad address 

我不确定哪里出了问题。如果我的 createPipe 方法有问题，或者如果我的 execute 方法有问题，或者我没有在我的 main 中正确处理事情程序。另一种意见真的很有帮助。

这是我的代码:

/* main.c */

#define NUM_ARGS 10
#define BUF_SIZE 128

void utility(char *array[]);

void cleararray(char *array[])
{
    int i=0;
    while(array[i]!=NULL)
    {
        array[i]=NULL;
        i++;
    }
}

int main(void){

    int redir_Flag=0;
    int i=0,j;
    char str[BUF_SIZE];
    char *p;
    char *array[NUM_ARGS];

    do
    {
        cleararray(array);
        printf("~MyShell $:");
        i=0;
        fgets(str,sizeof(str),stdin);
        p=strtok(str," \n");
        while(p!=NULL)
        {
            array[i]=(char*)malloc(sizeof(p));
            strcpy(array[i],p); 
            p=strtok (NULL," \n");
            i++;
            array[i]=NULL;
        }

        for(j=0;j<i;j++){
            if(strcmp(array[j],">")==0)
            {
                array[j]=NULL;
                redirect(array,array[j+1]);
                redir_Flag=1;
                break;
            }
            //printf("%s %d\n",array[j],j);
       }
       //printf("Before flag checking :%s\n",array[0]);
       /////////////////////////////////////////////  beginning of pipe check
       int q;
       for(q=0; q < i; q++){
           if(strcmp(array[q], "|") == 0)
           {
               printf("%s %s\n", array[q-1], array[q+1]);
               //char **cmda=NULL;
               //char **cmdb=NULL;
               createPipe(array[q-1], array[q+1]);
           }
       }
       if(redir_Flag==1)
       {
           redir_Flag=0;
           continue;
       }
       else{}
       if(array[0]==NULL){continue;}
       else if(strcmp(array[0],"exit")==0)
       {
           break;
       }
       else if(strcmp(array[0],"cd")==0)
       {
           change_Dir(array[1]);
       }
       else if(strcmp(array[0],"path")==0)
       {
           if(array[1]==NULL)
           {
               print_path();
           }
           else if(strcmp(array[1],"+")==0)
           {
               add_path(array[2]);
           }
           else if(strcmp(array[1],"-")==0)
           {
               remove_path(array[2]);
           }
           else {printf("invalid command\n");}
       }
       else if(array[0]==NULL){continue;}
       else
       {
           utility(array);
       }
       i=0;
       fflush(stdout);
       cleararray(array);
   }while(1);
   return 0;
}

管道.c

void createPipe(char **cmd1, char **cmd2){

    pid_t pid1; 
    pid_t pid2;

    int newPipefd[2];

    if(pipe(newPipefd)== -1){
        perror("pipe error");
        exit(1);
    }
    pid1 = fork();
    if(pid1 == -1){
        perror("fork error");
    }
    else if(pid1==0){
        //first child process closes the read end dup write
        close(1);
        dup(newPipefd[1]);
        close(newPipefd[1]);
        close(newPipefd[0]);

        execute(cmd1);     ////////////////////////here is where I have issues
        execv(cmd1[0], cmd1);

        perror("no exec child1");
    }
    else{
        pid2 = fork();
        if(pid2 ==-1){
            perror("fork error");
    }
    else if (pid2 == 0){
        //second child process closes up write side of pipe and dup read
        close(0);
        dup(newPipefd[0]);
        close(newPipefd[0]);
        close(newPipefd[1]);

        execute(cmd2);
        execv(cmd2[0], cmd2);
    } else {
        //parent
        int status;
        close(newPipefd[0]);
        close(newPipefd[1]);
        waitpid(pid1, &status, WUNTRACED | WCONTINUED);
        waitpid(pid2, &status, WUNTRACED | WCONTINUED);
    }
}  

执行方法

void execute(char *arg[]){
    int j=0,k;
    char *p;
    char *array[NUM_ENV];
    if(access(arg[0],F_OK)==0)
    {
        execv(arg[0],arg);
    }
    else
    {
        p=strtok(path,":\n");
        while(p!=NULL)
        {
            array[j]=(char*)malloc(sizeof(p));
            strcpy(array[j],p);
            p=strtok (NULL,":\n");
            printf("%s\n",array[j]);
            j++;
            array[j]=NULL;
        }
        for(k=0;k<j;k++){
            printf("in for loop : %s\n",arg[k]);
            strcat(array[k],"/");
            strcat(array[k],arg[0]);
            printf("IN FOR LOOP, ARRAY IS :%s\n",array[k]);

            if(access(array[k],F_OK)==0)
            {
                execv(array[k],arg);
            }
        }
        printf("Invalid command\n");
        fflush(stdout);
    }
}

Answer 1

我可以看出您的代码中存在一些问题。首先，这是一个错误:

 array[j]=(char*)malloc(sizeof(p));
 strcpy(array[j],p);

您为指针分配空间，而不是为字符串的副本分配空间。将这些行替换为:

 array[j] = strdup(p);

在 for 循环中，您不能这样做:

 strcat(array[k],"/");
 strcat(array[k],arg[0]);

因为 array[k] 没有额外的空间来放置最后的字符串。你应该重新分配 array[k]，这意味着你需要分配所有参数，这样你就不会最终泄漏内存，也不会混合分配的参数和未分配的参数，尽管这可能并不重要，因为你快速调用执行。

可能还有更多问题...