C语言简易Shell管道实现异常：执行ls | wc -l无输出

核心问题分析 #

1. 变量名笔误导致逻辑混乱 #

你用pipepid = fork();接收fork返回值，但后续判断用的是未定义的pid变量，直接导致子进程、父进程的执行逻辑完全偏离预期。

2. 管道文件描述符未完全关闭 #

• 子进程仅关闭管道写端pipe1[1]，未关闭无用的读端pipe1[0]，会造成管道资源泄漏，还可能让后续进程误判管道状态。
• 父进程仅关闭管道读端pipe1[0]，未关闭无用的写端pipe1[1]，这会让wc命令一直等待输入（管道写端未完全关闭，系统认为还有数据可能写入），最终无法输出结果。

3. 命令参数数组未以NULL结尾 #

execvp要求参数数组必须以NULL作为结尾标记，你拆分后的argsbefore和argsafter未添加该终止符，会导致execvp执行失败。

修正后的代码片段 #

int pipe1[2];
int pipepid;
int piping; /*flag for piping*/
int pipeposition;/*index of pipe indicator*/
//* code... */
if(pipe(pipe1)!= 0){
    perror("pipe");
    exit(1);
};
/* split commands to before pipe indicator and after */
for(int p = 0;p<pipeposition;p++){
    argsbefore[p]=args[p];
}
// 为参数数组添加NULL终止符
argsbefore[pipeposition] = NULL;
/* after */
int e=0;
for(int h = pipeposition+1; h<cnt;h++){
    argsafter[e]=args[h];
    e++;
}
// 为参数数组添加NULL终止符
argsafter[e] = NULL;
/* code ... */
if(piping){
    pipepid = fork();
    // 修正变量名，使用正确的fork返回值变量判断
    if(pipepid == 0){
        /* 子进程处理管道前命令 */
        // 关闭不需要的管道读端
        close(pipe1[0]);
        if(dup2(pipe1[1],1)==-1){
            perror("dup2 child");
            exit(1);
        }
        close(pipe1[1]);
        if (execvp(argsbefore[0], argsbefore) < 0) { 
            printf("exec failed\n");
            exit(1); 
        }
        exit(0);
    }/* fork失败处理 */
    else if(pipepid == -1){
        printf("ERROR: fork failed\n");
        exit(1);
    }/* 父进程处理管道后命令 */
    else{
        // 关闭不需要的管道写端
        close(pipe1[1]);
        if(dup2(pipe1[0],0)==-1){
            perror("dup2 parent");
            exit(1);
        }
        close(pipe1[0]);
        if (execvp(argsafter[0], argsafter) < 0) { 
            printf("exec failed\n");
            exit(1);
        } 
    }
}

额外提示 #

当前代码中父进程直接执行execvp会替换自身进程，执行完wc后Shell会直接退出。如果希望Shell继续运行，父进程应该再fork一个子进程处理管道后的命令，原Shell父进程等待两个子进程结束后继续执行主循环。

内容的提问来源于stack exchange，提问作者badcoding