当我使用sigaction时,为什么我的程序会出现意外行为?

时间:2016-05-07 20:59:30

标签: c linux signals fork posix

我正在编写自己的简单shell作为练习。我需要注册SIGCHLD信号才能处理僵尸进程。出于某种原因,当我使用sigaction添加处理程序时程序退出,我不明白为什么。

您可以在main()中看到我们退出process_arglist()返回0但我返回1并且我看不到信号处理会如何影响。{ / p>

这是我的代码。它应该处理以&结尾的命令(我们fork()并在子代码中使用execvp)。 例如:ping 127.0.0.1 -c 5 &

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <errno.h>
#include <pthread.h>
#include <signal.h>
#include <sys/wait.h>
#include <stdlib.h>
#include <unistd.h>

void sigchld_handler(int signal) {
  int my_errno = errno;
  while (waitpid(-1, 0, WNOHANG) > 0); // WNOHANG so handler will be non-blocking.
  errno = my_errno;
}

int handle_background_command(int count, char** arglist) {

  pid_t pid;
  arglist[count - 1] = NULL; // remove "&" from arglist

  //Handle SIGCHLD signal
  struct sigaction sa, sa_dft;
  sa.sa_handler = sigchld_handler;
  sa.sa_flags = SA_NOCLDSTOP;

  if (sigaction(SIGCHLD, &sa, &sa_dft) == -1) {
    perror("error when trying to set signal action");
    exit(-1);
  }

  if((pid = fork()) == -1) {
    perror("error when trying to fork() from handle_background_command()");
    exit(1);
  }

  if(pid == 0) {

    // Child code

    sigaction(SIGCHLD, &sa_dft, NULL);

    if(execvp(arglist[0], arglist) == -1) {
      perror("error when trying to execvp() from handle_background_command()");
      exit(1);
    }

  }

  // Parent code
  return 1;
}


int process_arglist(int count, char** arglist)
{
  return handle_background_command(count, arglist);
}

int main(void)
{
    while (1)
    {
        char** arglist = NULL;
        char* line = NULL;
        size_t size;
        int count = 0;

        if (getline(&line, &size, stdin) == -1) {
            printf("out!");
            break;
        }


        arglist = (char**) malloc(sizeof(char*));
        if (arglist == NULL) {
            printf("malloc failed: %s\n", strerror(errno));
            exit(-1);
        }
        arglist[0] = strtok(line, " \t\n");

        while (arglist[count] != NULL) {
            ++count;
            arglist = (char**) realloc(arglist, sizeof(char*) * (count + 1));
            if (arglist == NULL) {
                printf("realloc failed: %s\n", strerror(errno));
                exit(-1);
            }

            arglist[count] = strtok(NULL, " \t\n");
        }

        if (count != 0) {
            int result = process_arglist(count, arglist);
            printf("result = %d\n", result);
            if (!result) {
                free(line);
                free(arglist);
                printf("out\n");
                break;
            }
        }

        free(line);
        free(arglist);
    }
    pthread_exit(NULL);
    return 0;
}

同样,如果我摆脱了信号处理代码,那么它就可以了 什么原因?

修改 这是strace实用程序的输出(最后一行):

--- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=2818, si_status=0, si_utime=0, si_stime=0} ---
wait4(-1, NULL, WNOHANG, NULL)          = 2818
wait4(-1, NULL, WNOHANG, NULL)          = -1 ECHILD (No child processes)
rt_sigreturn()                          = -1 EINTR (Interrupted system call)
write(1, "out!", 4out!)                     = 4
exit_group(0)                           = ?
+++ exited with 0 +++

1 个答案:

答案 0 :(得分:1)

你的程序从getline函数退出EINTR(中断系统调用)(getline被信号处理程序中断)。

请检查:How to handle EINTR (interrupted System Call)