1

「CTRL-C」信号の処理に問題があります。必要なのは、handle_SIGINTを1回だけ出力してから、セットアップ関数の読み取りに戻すことですが、どうすればよいかわかりません。これ。readはブロッキング呼び出しであるため、length=read呼び出しの周りにループを追加するように言われました。これらの信号を処理する方法について混乱しています。とにかく私の現在の出力は次のとおりです。

COMMAND->test
COMMAND->test2
COMMAND->test3
COMMAND->test4
COMMAND->^CFound your CTRL-C
Found your CTRL-C
COMMAND->Found your CTRL-C
COMMAND->Found your CTRL-C
COMMAND->error reading the command: Interrupted system call
^Z
Suspended


#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <signal.h>

#define MAX_LINE 80 /* 80 chars per line, per command, should be enough. */

void handle_SIGINT()
{
  printf("Found your CTRL-C\n");
}

void setup(char inputBuffer[], char *args[],int *background)
{
    int length, /* # of characters in the command line */
         i,      /* loop index for accessing inputBuffer array */
         start,  /* index where beginning of next command parameter is */
        ct;     /* index of where to place the next parameter into args[] */

     ct = 0;

     /* read what the user enters on the command line */
     length = read(STDIN_FILENO, inputBuffer, MAX_LINE);  

     start = -1;
     if (length == 0)
        exit(0);            /* ^d was entered, end of user command stream */
     if (length < 0){
        perror("error reading the command");
    exit(-1);           /* terminate with error code of -1 */
    }

     /* examine every character in the inputBuffer */
     for (i = 0; i < length; i++) { 
        switch (inputBuffer[i]){
        case ' ':
        case '\t' :               /* argument separators */
            if(start != -1){
                args[ct] = &inputBuffer[start];    /* set up pointer */
                ct++;
            }
            inputBuffer[i] = '\0'; /* add a null char; make a C string */
            start = -1;
            break;

        case '\n':                 /* should be the final char examined */
            if (start != -1){
                args[ct] = &inputBuffer[start];     
                ct++;
            }
            inputBuffer[i] = '\0';
            args[ct] = NULL; /* no more arguments to this command */
            break;

        case '&':
            *background = 1;
            inputBuffer[i] = '\0';
            break;

        default :             /* some other character */
            if (start == -1)
                start = i;
    } 
    }    
    args[ct] = NULL; /* just in case the input line was > 80 */
} 

int main(void)
{
    char inputBuffer[MAX_LINE]; /* buffer to hold the command entered */
    int background;             /* equals 1 if a command is followed by '&' */
    char *args[MAX_LINE/2+1];/* command line (of 80) has max of 40 arguments */

    struct sigaction handler;
    handler.sa_handler = handle_SIGINT;
    sigaction(SIGINT, &handler, NULL);

    while (1){            /* Program terminates normally inside setup */
        background = 0;
    printf("COMMAND->");
        fflush(0);
        setup(inputBuffer, args, &background);       /* get next command */

        int pid;

    pid = fork(); /* for a child process */

    if (pid < 0) { /* check if error occurred with child process */
      fprintf(stderr, "Fork Failed");
      return 1;
    } else if (pid == 0) { /* child process */
      execvp(args[0], args);
    } else {
      if (background == 0){ /* check if the parent should wait */
        //wait(args);
        waitpid(pid, NULL, 0);
      }
    }

    /* the steps are:
      (1) fork a child process using fork()
      (2) the child process will invoke execvp()
      (3) if background == 0, the parent will wait, 
        otherwise returns to the setup() function. */
    }
}
4

5 に答える 5

3

read() が -1 を返した場合、チェックerrno- そうでない場合はEINTR、もう一度 read を実行します。 EINTRブロッキング システム コール (この場合は read()) がシグナルによって中断されていることを示します。

   EINTR  The call was interrupted by a signal before any data was read; see signal(7).

反対側から見ると、多くの場合、CTRL+C を押すと、ユーザーはプログラムを終了したいことを示しているため、プログラムを終了することは予期された動作です。

于 2012-10-19T20:09:18.493 に答える
1

メッセージを出力した後、handle_SIGINT の SIGINT シグナルを無視するようにプログラムに指示します。

void handle_SIGINT()
{
  printf("Found your CTRL-C\n");
  signal(SIGINT, SIG_IGN);
}
于 2012-10-19T20:11:50.333 に答える
0

これはあなたが使用して書くべきだった方法ですsigaction()：

struct sigaction new_handler,old_handler;

new_handler.sa_handler=handle_SIGINT;
sigemptyset(&new_handler.sa_mask);
new_handler.sa_flags=0;

sigaction(SIGINT,NULL, &old_handler);
if(old_handler.sa_handler != SIG_IGN)
   sigaction(SIGINT, &new_handler,NULL);
于 2012-10-19T20:47:22.083 に答える
0

推奨事項として、プログラムを可能な限り移植可能にするようにしてください。つまり、ANSI C を使用して大量のプログラムを作成できるように思えます。

read() の代わりに fgets() を使用することを検討し、bimda が指摘するようにその動作が十分に文書化されていても、決して動作に依存しないでください。

void setup(char inputBuffer[], char *args[],int *background)
{
    char *ret; 
    int length; /* # of characters in the command line */
    int i;      /* loop index for accessing inputBuffer array */
    int start=-1;  /* index where beginning of next command parameter is */
    int ct=0;     /* index of where to place the next parameter into args[] */


     /* read what the user enters on the command line */
     ret = fgets(inputBuffer, MAX_LINE, stdin);  

     if(!ret)
        exit(0);            /* ^d was entered, end of user command stream */

    /* examine every character in the inputBuffer */
    for (i = 0; inputBuffer[i]; i++) {
            ...
    }
    ...
}

パタターが言ったように、Cc部分：

void handle_SIGINT()
{
    printf("Found your CTRL-C\n");
    signal(SIGINT, SIG_IGN);
}

~$ 男シグナル

多分それはあなたが必要とするものではないかもしれませんが、それは私の謙虚な意見です.

于 2012-10-19T23:07:05.547 に答える