C语言和sigintHandler

Question

当我突然单击“ ctrl + c”时，我想删除FIFO文件。 我想捕获该信号，然后在真正终止进程之前删除after。 这是我的代码，我不知道出了什么问题：

 #include <fcntl.h>
 #include <stdio.h>
 #include  <stdlib.h>
 #include <sys/stat.h>
 #include <unistd.h>
 #include <signal.h>
 #define MAX_BUF 512

 void sigintHandler(int sig_num,char * myfifo)
{
puts("hello you there .please don't leave ");
unlink(myfifo);
exit(0);
}

int main()
{

printf("Access-Control-Allow-Origin: *\n");
printf("Content-Type: text/event-stream\n\n");
int fd;
char buf[MAX_BUF];

char * myfifo = "/tmp/omcipipe";
struct stat st;

// catch ctrl c

 sigaction(SIGINT, sigintHandler);



  //Create FIFO file 
     if (stat(myfifo, &st) != 0)
     mkfifo(myfifo, 0666);

 fd = open(myfifo, O_RDONLY);
while (read(fd, buf, MAX_BUF)>0)
{
    printf("data: %s", buf);
    printf("\n");
    fflush(stdout);
} 

puts( "----closing----"); 
close(fd);
unlink(myfifo);
return 0;

}

Answer 1

首先，您使用sigaction()设置信号处理程序的方式不正确，因为您没有填充struct sigaction所有成员。 填写所有必需的成员

struct sigaction info;
info.sa_handler = sigintHandler;
sigaction(SIGINT,&info,NULL);

其次，由于一次读取MAX_BUF ，因此不需要从fifo文件中读取数据， 而无需循环 。 不需要循环，像这样读取

int ret = read(fd, buf, MAX_BUF);
buf[ret-1] = '\0'; /*it should be null terminated */ 

第三， sigintHandler()仅包含一个参数。 从sigaction()的手册页

struct sigaction {
        void     (*sa_handler)(int);  /*handler expects only 1 argument */
        /* other member */
}

最后，最重要的是，从信号处理程序中调用诸如printf()和exit()之类的printf()并不安全。

您的sigHandler()看起来像

static void sigintHandler(int sig_num) {
        write(1,"in isr\n",strlen("in isr\n"));
        /*  set some flag variable here & use that 
        flag variable in main() function to remove the fifo */
}

看到这个如何避免在信号处理程序中使用printf？

这是示例代码

static int fifo_flag = 0;
static void sigintHandler(int sig_num) {
        write(1,"in isr\n",strlen("in isr\n"));
        fifo_flag = 1;
}
int main(void){
        printf("Access-Control-Allow-Origin: *\n");
        printf("Content-Type: text/event-stream\n\n");
        int fd = 0, index = 0;
        char buf[MAX_BUF];
        #if 0
        char *myfifo = "data";
        #endif
        //char * myfifo = "/tmp/omcipipe";
        struct stat st;
        struct sigaction info;
        info.sa_handler = sigintHandler;
        //info.sa_flags = /* set to defaulgs a/c to your requirement*/

        if (stat(myfifo, &st) != 0) {
                mkfifo(myfifo, 0666);
                perror("mkfifo");
        }
        fd = open(myfifo, O_RDONLY | 0666);
        if(fd == -1){
                perror("open");
                return 0;
        }
        char ch = 0;
        while(read(fd, &ch, 1) > 0) {
                sigaction(SIGINT,&info,NULL);/* if ctrl+c  is called */
                buf[index] = ch;
                //sleep(1);/* just to observe ctrl+c behaviour, not advised to use */
                printf("%c\n",buf[index]);
                index++;
        }
        buf[index] = '\0';
        printf("data: %s", buf);
        printf("\n");
        puts( "----closing----");
        close(fd);
        if(fifo_flag == 1) { /*if flag is set, unlink fifo */
                unlink(myfifo); /* you can unlink fifo file here */
        }
        return 0;
}

Answer 2

您的代码有很多问题。

此代码假定read()以'\\0'字符终止数据：

while (read(fd, buf, MAX_BUF)>0)
{
    printf("data: %s", buf);
    printf("\n");
    fflush(stdout);
}

read()仅读取原始字节，仅此而已。 它不能正确地以'\\0'字符终止字符串。 因此， printf()几乎肯定会调用未定义的行为。

这段代码

sigaction(SIGINT, sigintHandler);

是错的。 sigaction()具有三个参数 ：

   #include <signal.h>

   int sigaction(int signum, const struct sigaction *act,
                 struct sigaction *oldact);

您还只能从信号处理程序中进行异步信号安全的函数调用 。 这段代码

 void sigintHandler(int sig_num,char * myfifo)
{
puts("hello you there .please don't leave ");
unlink(myfifo);
exit(0);
}

调用puts()和exit() ，两者都不是异步信号安全的。

并且如注释中所述，信号处理程序代码假定将myfifo作为第二个参数传递。 不是。 在sigaction手册页中 ，第二个参数是struct siginfo * ，其中包含有关信号上下文的信息-但仅当传递给注册信号处理程序的sigaction()调用的struct sigaction设置了SA_SIGINFO标志，并且处理程序集使用sa_sigaction成员而不是sa_handler成员。