为何含execlp的while循环仅执行一次迭代就退出？

核心错误点 #

• 全局变量child_pid无效：fork后父子进程拥有独立的地址空间，子进程对child_pid的赋值不会同步到父进程。父进程里的child_pid是未初始化的垃圾值，用它调用kill要么无效，要么会误杀其他进程。
• kill操作完全多余：wait(NULL)已经等待子进程退出，退出后的进程会被wait回收资源，此时子进程已不存在，kill没有任何意义。
• 子进程未处理execlp失败的情况：如果execlp调用失败（比如命令不存在），子进程会继续执行后续代码，进入循环接收用户输入，导致多个进程同时等待输入，程序逻辑彻底混乱。

修正后的代码 #

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/wait.h>
#include <signal.h>

int counter = 0;

int main()
{
    char c;

    while(1)
    {
        printf("enter choice: ");
        // 处理输入缓冲区残留的换行，避免后续读取异常
        if (scanf(" %c", &c) != 1) {
            while(getchar() != '\n');
            continue;
        }

        if (c == 'c')
        {
            pid_t id = fork();
            if (id == 0){
                printf("\ncreated child process at PID: %d\n", getpid());
                // execlp成功后不会返回，失败才会执行后续代码
                execlp("ls", "ls", "-la", (char *)0);
                // 走到此处说明execlp调用失败
                perror("execlp failed");
                exit(EXIT_FAILURE);
            }
            else if (id > 0){
                // 等待子进程退出并回收资源
                wait(NULL);
                printf("\nchild process exited\n");
                counter +=1;
            }
            else{
                perror("fork failed");
            }
        }
        else if (c == 'e')
        {
            exit(EXIT_SUCCESS);
        }
    }

    return 0;
}

关键修正说明 #

• 移除全局child_pid，直接使用fork返回值：父进程中fork的返回值就是子进程的PID，无需子进程传递，从根源避免了地址空间隔离带来的变量不同步问题。
• 删除多余的kill调用：wait(NULL)会完成子进程的资源回收，子进程退出后已不存在，无需额外杀死操作。
• 给子进程添加execlp失败后的退出逻辑：一旦execlp调用失败，立即打印错误信息并退出子进程，防止子进程执行父进程的循环逻辑。
• 增加输入异常处理：当输入不符合预期时，清除输入缓冲区残留内容，避免后续读取陷入死循环。

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