使用 SIGCHLD 处理僵尸

Question

在我的程序中，我正在监听传入的 SIGCHLD 信号以避免僵尸。

代码：

void myhandler(int signo)
{   
    printf("test");
    int status;
    pid_t pid;

    while((pid = waitpid(-1, &status, WNOHANG)) > 0)
        ++count;
}

int main(int argc, char const *argv[])
{

    struct sigaction sigchld_action;
    memset(&sigchld_action,0,sizeof(sigchld_action));
    sigchld_action.sa_handler = &myhandler;
    sigaction(SIGCHLD,&sigchld_action,NULL);

    if(fork() == 0){
        exit(0);
    }
    if(fork()==0){
        exit(0);
    }
    if(fork()==0){
        exit(0);
    }


    while(wait(NULL) > 0)
    ++count;

    return 0;
}

问题是，分叉子的数量和 printf("test") 的输出数量有时不匹配。分叉子的数量大于数量 printf("test")。

这段代码是否保证不会有僵尸？如果是，它是如何做到这一点的？它没有打印正确数量的“测试”。 waitpid() 是否在 while 内多次清除死去的孩子？

当这个信号处理程序被调用时会发生什么，同时另一个孩子可能会死掉。默认情况下，信号将被阻止。（当处理程序运行时，另一个孩子可能会死亡）。 waitpid 是否清除信号处理程序运行时发送的信号的进程？

另外，计数器不相等。 (static volatile int) 或者我尝试了原子整数。

Answer 1

正在发生两种不同的事情：

1. printf 不是异步信号安全的，因此您不应从信号处理程序中调用它。将其替换为write。
2. 您只在信号处理程序中增加count 一次。如果您想知道有多少进程死亡，则需要在 while 循环中增加它。

试试这个代码：

void handler(int signo)
{   
    int status;
    pid_t pid;
    while((pid = waitpid(-1, &status, WNOHANG)) > 0) {
        write(1, "test", 4); /* technically this may result in a partial write and you should loop it, but in practice I think this'll be fine for this example */
        ++count;
    }
}

使用该代码，您可能没有僵尸，并且tests 的正确数量和count 中的正确值。但是，还有一个竞争条件：如果在对waitpid 的最终调用和信号处理程序结束之间，另一个子进程死亡，那么将不会收到SIGCHLD，因此它将是一个僵尸直到它之后的那个也死了。这种极端情况的解决方案要复杂得多，并且取决于应用程序其余部分的结构。