我的信号处理程序中的竞争条件？ （C）

Question

我正在为系统课程的 shell 实验室工作，自周五晚上以来我一直在尝试解决一些非常奇怪的竞态条件错误，但似乎无法确定。

我当前的代码：http://buu700.com/tsh

START OF MY CODE 之前和END OF MY CODE 之后的所有内容均由课程讲师提供，因此这些都不应该是问题的根源。

我们也有一个测试脚本；这是我当前测试结果的输出：http://buu700.com/sdriver

---

/*****************
 * Signal handlers
 *****************/

/* 
 * sigchld_handler - The kernel sends a SIGCHLD to the shell whenever
 *     a child job terminates (becomes a zombie), or stops because it
 *     received a SIGSTOP, SIGTSTP, SIGTTIN or SIGTTOU signal. The 
 *     handler reaps all available zombie children, but doesn't wait 
 *     for any other currently running children to terminate.  
 */
void 
sigchld_handler(int sig) 
{
    pid_t pid;
    int status, termsig;
    struct job_t *job;


    sigset_t s;

    sigemptyset(&s);
    sigaddset(&s, SIGCHLD);
    sigaddset(&s, SIGINT);
    sigaddset(&s, SIGTSTP);

    sigprocmask(SIG_BLOCK, &s, NULL);


    while ((pid = waitpid(-1, &status, WNOHANG | WUNTRACED)) > 0) {
        if (WIFEXITED(status)) {
            deletejob(job_list, pid);
        }
        if ((termsig = WTERMSIG(status))) {
            deletejob(job_list, pid);
            safe_printf("Job [%i] (%i) %s by signal %i\n",
                            pid2jid(pid), pid, "terminated", termsig);
        }
        if (WIFSTOPPED(status)) {
            job = getjobpid(job_list, pid);
            job->state = ST;
            safe_printf("Job [%i] (%i) %s by signal %i\n",
                            pid2jid(pid), pid, "stopped", SIGTSTP);
        }
    }

    if (errno != ECHILD)
        unix_error("waitpid error");

    sigprocmask(SIG_UNBLOCK, &s, NULL);

    return;
}



/* 
 * sigint_handler - The kernel sends a SIGINT to the shell whenver the
 *    user types ctrl-c at the keyboard.  Catch it and send it along
 *    to the foreground job.  
 */
void 
sigint_handler(int sig) 
{
    sigset_t s;

    sigemptyset(&s);
    sigaddset(&s, SIGCHLD);
    sigaddset(&s, SIGINT);
    sigaddset(&s, SIGTSTP);

    sigprocmask(SIG_BLOCK, &s, NULL);

    kill(-1, sig);

    sigprocmask(SIG_UNBLOCK, &s, NULL);

    return;
}



/*
 * sigtstp_handler - The kernel sends a SIGTSTP to the shell whenever
 *     the user types ctrl-z at the keyboard. Catch it and suspend the
 *     foreground job by sending it a SIGTSTP.  
 */
void 
sigtstp_handler(int sig) 
{
    sigset_t s;

    sigemptyset(&s);
    sigaddset(&s, SIGCHLD);
    sigaddset(&s, SIGINT);
    sigaddset(&s, SIGTSTP);

    sigprocmask(SIG_BLOCK, &s, NULL);

    kill(-1, sig);

    sigprocmask(SIG_UNBLOCK, &s, NULL);

    return;
}

Answer 1

我怀疑该错误来自您与信号处理程序中的其他信号竞争：

sigint_handler(int sig) 
{
    sigset_t s;

    sigemptyset(&s);
    sigaddset(&s, SIGCHLD);
    sigaddset(&s, SIGINT);
    sigaddset(&s, SIGTSTP);

    sigprocmask(SIG_BLOCK, &s, NULL);

当您向sigaction(2) 注册信号处理程序时，您可以提供一个sa_mask，内核将在您的信号处理程序运行时为您阻止信号。这是原子完成的，不需要您做额外的工作。注册信号处理程序时只需填充一次此掩码。

另一种可能性来自SIGTSTP信号；我的APUE, 2nd edition 副本的第 350 页部分说：

只有作业控制 shell 才能将 [SIGTSTP, SIGTTIN, SIGTTOU] 的配置重置为 SIG_DFL。

那些段落中没有说的，但我认为可以公平地假设是，shell 应该为 子进程将信号处置设置为SIG_DFL——它仍然需要做一些事情自己处理信号。

您是否妥善处理了孩子们的信号处置？