Dlaczego execvp() wykonuje dwukrotnie używając fork()?

Question

Wdrażam powłokę.

Po uruchomieniu polecenia innego niż zmiana katalogu, execvp() działa, dziecko kończy działanie i tworzone jest nowe dziecko. Kiedy zmieniam katalogi, dziecko się nie kończy i powstaje nowe dziecko. Oto przykład z mojego kodu:

for(;;) { 
    printf("bash: "); 
    parse(); 
    ... 
    pid_t pid = fork() 
    if (pid == 0) 
     if (!strcmp(line[0], "cd")) 
      if (!line[1]) (void) chdir(getenv("HOME")); 
      else (void) chdir(line[1]); 
     else execvp(line[0], line); 
    ... 
    if (pid > 0) { 
     while (pid == wait(NULL)); 
     printf("%d terminated.\n", pid); 
    } 
} 

cd ../; ls; działa poprawnie, z wyjątkiem muszę Ctrl+D dwukrotnie, aby zakończyć program.

Mimo to, jeśli wypiszę te same informacje (np. mybash < chdirtest), to działa poprawnie raz, kończy dziecko, ponownie działa, z wyjątkiem oryginału bezpośrednio, a następnie kończy ostatnie dziecko.

Answer 1

cd nie należy wywoływać przez proces potomny, sama powłoka powinna zmienić swój bieżący katalog (jest to właściwość wewnętrznego polecenia: modyfikować proces samej powłoki).

A (primitve) powłoka powinna wyglądać tak:

for(;;) { 
    printf("bash: "); 
    parse(); 

    // realize internal commands (here "cd") 
    if (!strcmp(line[0], "cd")) { 
     if (!line[1]) (void) chdir(getenv("HOME")); 
     else (void) chdir(line[1]); 
     continue; // jump back to read another command 
    } 

    // realize external commands 
    pid_t pid = fork() 
    if (pid == 0) { 
     execvp(line[0], line); 
     exit(EXIT_FAILURE); // wrong exec 
    } 

    // synchro on child 
    if (pid > 0) { 
     while (pid == wait(NULL)); 
     printf("%d terminated.\n", pid); 
    } 
}