フォークと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 

であり、たとえば、その日のメッセージを表示します単にprintステートメントだけです。今後のアドバイスはありますか？

Answer 1

示されたコードには複数のバグがあります。それの額面で

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

argvNewは、だから、配列は過去オフ値argvNew[argc]通じargvNew[0]、およびargvNew[argc+1]=NULL;の実行が含まれてい

char *argvNew[argc + 1]; 

として宣言されているため、NULLの割り当てが間違っている、と未定義の動作になります配列の終わりで、結果として未定義の動作をします。これは明らかに

argvNew[argc] = NULL; 

でなければなりません。しかしのでそれさえも、間違って次のようになります。

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

argvNew[0]が実行されて、このプログラムの名前である、argv[0]からコピーされます。これは、子プロセスで同じプログラムをforkして実行します。

あなたは自分をforkbombingなってしまいます。これが共有サーバーの場合は、システム管理者に非常に怒ってしまいます。

式からargv[0]を削除して、argv[1]以降のみをコピーする必要があります。正しいループ、およびコピーは、次のとおりです。

int pid; 
    char *argvNew[argc]; 

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

Answer 2

execve()の呼び出しのパラメータは、実行するファイル名であることを最初のパラメータが必要です。残念ながらargvNew[0]にはと同じ値が渡されます。これは、自分のプログラムを呼び出して呼び出し、スクリプトを呼び出さないことを意味します。パラメータを1つずつシフトする必要があります。

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