1

我的问题是我无法理解为什么这些代码仍然会生成僵尸进程。我该如何解决?谢谢。

#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;
}
4

1 回答 1

3

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

您不会为每个退出的孩子传递一个信号。如果两个孩子“同时”退出,你只会得到一个SIGCHLD交付。因此,当您在信号处理程序(其中任何一个)中等待时,您应该循环等待以获取所有已退出的子节点。类似于以下内容:

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

当您的进程退出时,该进程的子进程将由 init 进程继承,因此它们将被正确收割。但是,您的无限循环阻止了 3 件事:

  • 它不会收获同时退出的孩子。
  • 在收获第一个孩子之后,它不会再收获任何孩子。
  • 它不允许进程退出,因此子进程不能被 init 进程收割。
于 2013-06-06T18:04:25.023 回答