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进程无法获得子进程。

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

1 个答案: