为什么这个程序会创建一个僵尸进程，我该如何解决？

Question

我的问题是我无法承认为什么这些代码仍会生成僵尸进程。 我怎么修好它？ 谢谢。

#include <signal.h>
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <time.h>


int count = 0;

/**
 * signal  callback
 * @param signo
 */
void sigcallback(int signo) {
    short stime = 5;
    int status;
    wait(&status);
    while (1) {

        printf("wake up,%d\n", getpid());
        sleep((stime + 1) - (short) (time(NULL) % stime));

    }
}

int call_cmd(void) {
    count++;

    int i;
    pid_t pid;
    signal(SIGCHLD, sigcallback);
    for (i = 0; i < 3; i++) {
        if ((pid = fork()) < 0) {
            printf("error with fork\n");
            exit(1);
        } else if (pid > 0) {
            waitpid(pid, NULL, WNOHANG);
        } else {
            printf("\t\t\tsid is %d,count is %d,pid is %d\n", getsid(getpid()), count, getpid());
        }
    }
    return 1;
}

void notice_root(int signo) {
    printf("execut root signal\n");
    waitpid(0, NULL, 0);
}

int main(void) {
    pid_t pid;

    int i;
    signal(SIGCHLD, notice_root);
    for (i = 0; i < 2; i++) {
        if ((pid = fork()) < 0) {
            printf("error with fork\n");
            exit(1);
        } else if (pid > 0) {
            exit(0);
        } else {
            setsid();
            call_cmd();
        }
    }
    return 0;
}

Answer 1

您的信号处理程序sigcallback有一个无限的睡眠循环。 一旦该函数处理了一个信号，该进程将永远不再处理另一个SIGCHLD信号，因为信号处理程序永远不会返回。 这可能是你的僵尸的原因。

您没有为每个退出的孩子提供一个信号。 如果两个孩子“同时”退出，您只能获得一个SIGCHLD 。 因此，当您在信号处理程序（其中任何一个）中等待时，您应该在循环中等待所有已退出的子项。 有点像：

for (;;) {
    pid_t p = waitpid(-1, 0, WNOHANG);
    if (p <= 0) break;
    /* ... */
}

当您的进程退出时，该进程的子进程将由init进程继承，因此它们将被正确地获取。 然而，你的无限循环阻止了三件事：

• 它不会同时收获退出的孩子。
• 在收获第一个孩子后，它不会再收获任何孩子了。
• 它不允许进程退出，因此init进程无法获得子进程。