c - 后台进程和挂起进程 - 在 C 中实现作业控制 Shell

Question

我正在 Linux 中用 C 实现作业控制 Shell 作为操作系统相关主题的项目。我有一个执行子进程管理的 main() 函数，通过链接列表提供帮助，如下所示，其中存储了后台和挂起的作业信息:

typedef struct job_
{
    pid_t pgid; /* group id = process lider id */
    char * command; /* program name */
    enum job_state state;
    struct job_ *next; /* next job in the list */
} job;

每次子进程退出或停止时，都会向父进程发送一个 SIGCHLD 以通知该情况。然后，我有一个信号处理程序，如此处所示，对于该作业状态链表的每个节点，检查该节点中表示的进程是否已退出，如果退出，则从链表中删除该节点。以下是 SIGCHLD 处理程序的代码，其中“job_list”是存储信息的链表:

void mySIGCHLD_Handler(int signum) {
    block_SIGCHLD();
    if (signum == 17) {
        job *current_node = job_list->next, *node_to_delete = NULL;
        int process_status, process_id_deleted;

        while (current_node) {

            /* Wait for a child process to finish.
            *    - WNOHANG: return immediately if the process has not exited
            */
            waitpid(current_node->pgid, &process_status, WNOHANG);

            if (WIFEXITED(process_status) != 0) {
                node_to_delete = current_node;
                current_node = current_node->next;
                process_id_deleted = node_to_delete->pgid;
                if (delete_job(job_list, node_to_delete)) {
                printf("Process #%d deleted from job list\n", process_id_deleted);
                } else {
                    printf("Process #%d could not be deleted from job list\n", process_id_deleted);
                }
            } else {
                current_node = current_node->next;
            }
        }
    }
    unblock_SIGCHLD();
}

问题是，当处理程序被调用时，一些不应该被删除的条目因为它们代表的进程没有退出，在它们不应该被删除的时候被删除了。谁知道为什么会这样？

谢谢你，很抱歉让你浪费时间:(

Answer 1

我看到这段代码中有很多问题，但最直接的问题可能在这里:

        waitpid(current_node->pgid, &process_status, WNOHANG);
        if (WIFEXITED(process_status) != 0) {

当waitpid(pid, &status, WNOHANG)返回时，因为进程还没有退出，所以不会向status写入任何东西，所以后面的if 在垃圾上分支。在假设 status 有意义之前，您需要检查 waitpid 的实际返回值。

其他最重要的问题是:

• 内核只允许发送一个SIGCHLD 来告诉您多个进程已经退出。当您收到 SIGCHLD 时，您需要在循环中调用 waitpid(0, &status, WNOHANG) 直到它告诉您没有更多的进程要等待，并且您需要处理(没有双关语意)所有它告诉您的已退出进程 ID。

• 从异步信号处理程序调用 printf 或 free 是不安全的。相反，将终止的进程添加到延迟任务列表中。确保在使用该列表的主循环代码中阻止 SIGCHLD。

• 不要在处理程序中自己阻止和取消阻止 SIGCHLD；有一个不可避免的竞争条件。相反，让内核通过正确设置信号处理程序以原子方式为您完成:使用 sigaction 并且不要将 SA_NODEFER 放入 sa_flags。 (务必将 SA_RESTART 放入 sa_flags 中，除非您有充分的理由不这样做。)

• 文字数字 17 应该是信号常量 SIGCHLD。 一些信号编号在整个历史上的所有 Unix 中都是稳定的，但 SIGCHLD 不是其中之一。