popen2로 만든 살인 프로세스

Question

나는 잠시 후 다시 죽여야 만하는 프로세스를 프로그래밍 방식으로 생성하기 위해 popen2 (stackoverflow의 다른 곳에서 권장되는) 함수를 사용하고 있습니다. popen2는 PID를 반환하고이 PID를 사용하여 프로세스를 종료 할 수 있다고 생각했습니다. 하지만이 방법으로는 작동하지 않습니다. 그것을 죽이려면 반환 된 PID를 증가시켜야하는데 이해할 수 없습니다 (아래 코드 참조)

다양한 스레드가 병렬로이를 수행 할 때 또 다른 문제가 발생할 수 있습니다. 이 경우 경쟁 조건으로 인해 PID가 둘 이상 다를 수 있다고 생각합니다.

내 질문은 : 어떻게 안정적으로 멀티 스레드 시나리오에서 popen2에 의해 만들어진 프로세스의 PID를 식별 할 수 있습니까?

#include <sys/types.h> 
#include <fcntl.h> 
#include <unistd.h> 
#include <signal.h> 

#define READ 0 
#define WRITE 1 

pid_t popen2(const char *command, int *infp, int *outfp) { 

    int p_stdin[2], p_stdout[2]; 
    pid_t pid; 

    if (pipe(p_stdin) != 0 || pipe(p_stdout) != 0) 
     return -1; 

    pid = fork(); 

    if (pid < 0) 
     return pid; 
    else if (pid == 0) 
    { 
     close(p_stdin[WRITE]); 
     dup2(p_stdin[READ], READ); 
     close(p_stdout[READ]); 
     dup2(p_stdout[WRITE], WRITE); 

     execl("/bin/sh", "sh", "-c", command, NULL); 
     perror("execl"); 
     exit(1); 
    } 

    if (infp == NULL) 
     close(p_stdin[WRITE]); 
    else 
     *infp = p_stdin[WRITE]; 

    if (outfp == NULL) 
     close(p_stdout[READ]); 
    else 
     *outfp = p_stdout[READ]; 

    return pid; 
} 

main() { 
    pid_t pid; 

    // Create process 
    pid = popen2("crafty", &in, &out); 

    sleep(5); 

    // Why doesn't kill(pid, SIGKILL) work? 
    kill(pid+1, SIGKILL); 

    while (1); 
} 

Answer 1

나는 그것을 얻었습니다.

execl("/bin/sh", "sh", "-c", command, NULL); 

실행 sh하고 PID의 popen2 돌아갑니다. kill에 전화하면 sh이 (가) 만료되지만 하위 프로세스 인 command에는 건드리지 않습니다. 그것은 실제로 다음 pid를 죽이는 우연히 command를 죽입니다. 이것은 항상 작동하지 않고 경주 조건에 달려 있습니다.

목표 프로세스를 종료하려면 직접 시작해야합니다.

경고 (테스트되지 않은 코드) :

pid_t popen2(const char **command, int *infp, int *outfp) { 

    int p_stdin[2], p_stdout[2]; 
    pid_t pid; 

    if (pipe(p_stdin) != 0 || pipe(p_stdout) != 0) 
     return -1; 

    pid = fork(); 

    if (pid < 0) 
     return pid; 
    else if (pid == 0) 
    { 
     close(p_stdin[WRITE]); 
     dup2(p_stdin[READ], READ); 
     close(p_stdout[READ]); 
     dup2(p_stdout[WRITE], WRITE); 

     execvp(*command, command); 
     perror("execvp"); 
     exit(1); 
    } 

    if (infp == NULL) 
     close(p_stdin[WRITE]); 
    else 
     *infp = p_stdin[WRITE]; 

    if (outfp == NULL) 
     close(p_stdout[READ]); 
    else 
     *outfp = p_stdout[READ]; 

    return pid; 
} 

및 특정 예에서

char *command[] = {"program", "arg1", "arg2", ..., NULL}; 

의 형태로 명령을 전달합니다

char *command[] = {"crafty", NULL}; 

Answer 2

당신이 사용할 수있는 '간부'명령 보류중인 프로세스를 피하기위한 쉘. 또한 popen2는 사용하지 않은 파이프의 쓰기 끝을 닫아야합니다. 그렇지 않으면 파이프가 열려 있습니다. 포인터 (infp, outpf) 중 하나가 NULL이면 파이프를 만들고 즉시 닫는 것은 쓸모가 없습니다. 다음은 프로젝트에서 사용하는 popen2 버전입니다.

pid_t popen2(char *command, int *in_fd, int *out_fd) { 
    int  pin[2], pout[2]; 
    pid_t pid; 
    char cmd[strlen(command)+10]; 

    if (out_fd != NULL) { 
     if (pipe(pin) != 0) return(-1); 
    } 
    if (in_fd != NULL) { 
     if (pipe(pout) != 0) { 
      if (out_fd != NULL) { 
       close(pin[0]); 
       close(pin[1]); 
      } 
      return(-1); 
     } 
    } 

    pid = fork(); 

    if (pid < 0) { 
     if (out_fd != NULL) { 
      close(pin[0]); 
      close(pin[1]); 
     } 
     if (in_fd != NULL) { 
      close(pout[0]); 
      close(pout[1]); 
     } 
     return pid; 
    } 
    if (pid==0) { 
     if (out_fd != NULL) { 
      close(pin[1]); 
      dup2(pin[0], 0); 
     } 
     if (in_fd != NULL) { 
      close(pout[0]); 
      dup2(pout[1], 1); 
     } 
     // Exec makes possible to kill this process 
     sprintf(cmd, "exec %s", command); 
     execlp("sh", "sh", "-c", cmd, NULL); 
     fprintf(stderr, "%s:%d: Exec failed in popen2. ", __FILE__, __LINE__); 
     perror("Error:"); 
     exit(1); 
    } 
    if (in_fd != NULL) { 
     close(pout[1]); 
     *in_fd = pout[0]; 
    } 
    if (out_fd != NULL) { 
     close(pin[0]); 
     *out_fd = pin[1]; 
    } 
    return pid; 
}