Question

我正在用C语言编写一个简单的shell，它支持执行普通的Linux命令以及我自己编写的一些内置命令（例如退出该shell时使用“ quit”）。 Shell为输入的每个命令派生一个新的子代。我有一个SIGCHLD处理程序，可在子进程完成时收获僵尸子程序。我的问题是，每当我运行后台进程时，调用SIGCHLD处理程序时，父进程（实际的shell程序）也会被杀死。

我已经看过this post并做了一个SIGPIPE处理程序，但是它从未被调用过，所以我不认为这是问题所在。

此eval函数处理键入的每个命令，并决定是在前台还是在后台运行该命令：

void eval(char *cmdline) 
{
    char *argv[MAXARGS];
    char buf[MAXLINE];
    int bg;
    pid_t pid;
    sigset_t mask_all, mask_one, prev_one;

    sigfillset(&mask_all);
    sigemptyset(&mask_one);
    sigaddset(&mask_one, SIGCHLD);


    strcpy(buf, cmdline);
    /* Is it a background command (ends in &) ? */
    bg = parseline(buf, argv);

    /*Ignore empty lines */
    if(argv[0] == NULL)
        return;

    /*builtin_cmd just handles my own custom commands */
    if(!builtin_cmd(argv)) {

        /*Block SIGCHLD signals */
        sigprocmask(SIG_BLOCK, &mask_one, &prev_one);
        if((pid = fork()) == 0) {
            /* Unblock SIGCHLD */
            sigprocmask(SIG_SETMASK, &prev_one, NULL);
            if(execve(argv[0], argv, environ) < 0) {
                fprintf(stderr, "%s: Command not found\n", argv[0]);
                fflush(stderr);
                exit(0);
            }
        }

        /* Block all signals */
        sigprocmask(SIG_BLOCK, &mask_all, NULL);
        setpgid(pid, 0);
        if(!bg) {
            /*Wait for foreground job to complete */
            int status;
            if(waitpid(pid, &status, 0) < 0) {
                fprintf(stderr, "wait pid error");
            }
        } else {
            /* Add the job to the jobs array */
            /* BG just means it's in the background */
            addjob(jobs, pid, BG, cmdline);
            int jid = (*getjobpid(jobs, pid)).jid;
            printf("[%d] (%d) %s", jid, pid, cmdline);
        }
        /*Unblock signals */
        sigprocmask(SIG_SETMASK, &prev_one, NULL);
    }

    return;
}

这是我的SIGCHLD处理程序：

void sigchld_handler(int sig) 
{
    int olderrno = errno;
    sigset_t mask_all, prev_all;
    sigfillset(&mask_all);

    /*Reap zombie */
    pid_t pid;
    if((pid = waitpid(-1, NULL, WNOHANG)) > 0) {
        sigprocmask(SIG_BLOCK, &mask_all, &prev_all);   //Block all signals
        deletejob(jobs, pid);                           //Delete job from jobs array
        sigprocmask(SIG_SETMASK, &prev_all, NULL);      //Unblock signals
    }

    //WHY DOES THIS EXECUTE FOR BACKGROUND JOBS???
    if(errno != ECHILD) {
        printf("waitpid error");
    }
    errno = olderrno;

    return;
}

waitpid杀死了孩子，这从我放入其中的打印语句可以知道。但是，当我运行后台进程时，父进程也会被杀死，并且会显示“ waitpid error”消息。

brett@brett-linux:~/Desktop/Shell$ ./tsh
tsh> /bin/sleep 1 &
[1] (7223) /bin/sleep 1 &
tsh> waitpid error
brett@brett-linux:~/Desktop/Shell$

如您所见，程序死了。为什么会这样？

Answer 1

删除非信号安全功能printf可以解决此问题。

杀死派生的子进程会杀死SIGCHLD处理程序中的父进程

1 个答案: