포크 및 execve 세그먼트 화 오류

Question

내 프로그램에서 fork 및 exec 시스템 호출을 사용해야합니다. exec는 자식 프로세스를 변경하여 다른 명령을 인수로 사용하고 해당 명령을 실행합니다. ./myexecv cat etc/motd 아무 일을 실행하지 후이 내 현재 코드

extern char **environ;  /* environment info */ 
main(int argc, char **argv) { 
    /* argc -- number of arguments */ 
    /* argv -- an array of strings */ 

    char *argvNew[argc + 1]; 
    int pid; 

    for(int i=0; i<argc; i++){ 
     argvNew[i] = argv[i]; 
    } 
    argvNew[argc + 1] = NULL; 
    printf("After For: %s\n",argvNew[0]); 
    printf("After For: %s\n",argvNew[1]); 
    printf("After For: %s\n",argvNew[2]); 
    printf("After For: %s\n",argvNew[3]); 


    if ((pid = fork()) < 0) { 
     fprintf(stderr, "Fork error%sstrerror\n", strerror(errno)); 

     exit(1); 
    } 
    else if (pid == 0) { 
     /* child process */ 
     if (execve(argvNew[0], argvNew, environ) < 0) { 
      fprintf(stderr, "Execve error%d %s\n",errno,strerror(errno)); 
      exit(1); 
     } 
    } 
    else { 
     /* parent */ 
    wait(0);  /* wait for the child to finish */ 
    } 

} 

이

./myexec cat /etc/motd 

입니다 : 예를 들어, 오늘의 메시지를 표시합니다 그냥 인쇄 문. 앞으로 어떤 조언이 필요합니까?

Answer 1

표시된 코드에 여러 가지 버그가 있습니다.

그것의 액면가에

for(int i=0; i<argc; i++){ 
      argvNew[i] = argv[i]; 
    } 
    argvNew[argc+1] = NULL; 

의 NULL 할당은 잘못이며, argvNew이

char *argvNew[argc + 1]; 

로 선언되어 있기 때문에 그래서 배열이 argvNew[0]argvNew[argc]을 통해 값을 포함하는 정의되지 않은 동작이 발생할 것이며, argvNew[argc+1]=NULL; 과거를 실행 정의되지 않은 동작을 초래하는 배열의 끝. 이것은 분명히

argvNew[argc] = NULL; 

해야하지만 때문에 심지어는 잘못된 것 :

execve(argvNew[0], argvNew, environ); 

argvNew[0]이 실행되는이 프로그램의 이름 인 argv[0]에서 복사됩니다. 이것은 하위 프로세스에서 동일한 프로그램을 포크하고 실행합니다.

당신은 자신을 위해 폭탄 테러를 당하게 될 것입니다. 공유 서버 인 경우 시스템 관리자를 아주 미친 듯이 만듭니다.

방정식에서 argv[0]을 제거하고 argv[1] 이상을 복사해야합니다. 정확한 루프 및 복사된다 : execve()의

int pid; 
    char *argvNew[argc]; 

    for(int i=1; i<argc; i++){ 
      argvNew[i-1] = argv[i]; 
    } 
    argvNew[argc-1] = NULL; 

Answer 2

발신 파라미터 실행할 파일명 최초 파라미터를 필요로한다. 불행히도 argvNew[0]을 전달하면 argv[0]과 같은 값입니다. 즉, 자신의 프로그램을 호출하고 다시 호출해야하며 스크립트는 절대 호출하지 말아야합니다. 매개 변수를 하나씩 이동해야합니다.

... 
for(int i=1; i<argc; i++){ 
    argvNew[i-1] = argv[i]; 
} 
argvNew[argc-1] = NULL; 
...