Простая оболочка Linux - execvp() не удалось

Question

Мне нужна помощь с простой оболочкой для класса, и я волнуюсь, я не совсем понимаю, как работает функция execvp().

Оболочка не делает много, не поддерживает трубопроводы, перенаправление, сценарии или что-то подобное. Он читает только команду, читает в параметрах (с командой как опция [0]) и вилки.

Он работал несколько раз, а затем начал давать мне ошибки о том, что вы не можете найти команды. Другие подобные вопросы, заданные здесь, касались трубопроводов или перенаправления.

Пожалуйста, простите noobcode, это не очень, но я надеюсь, что это разборчивым:

#include <stdio.h> 
#include <string.h> 
#include <unistd.h> 
#include <sys/wait.h> 
#include <sys/types.h> 

#define OPT_AMT 10 

const size_t SIZE = 256; 
int i = 0; 
int o = 0; 

int main(void) { 

    // initializing data 

    int exit = 0; 
    char cwd[SIZE]; 
    char cmd[SIZE]; 
    char input[SIZE * OPT_AMT]; 
    char *opt[OPT_AMT]; 
    getcwd(cwd, SIZE); 

    // main loop 

    while (exit == 0) { 
     // reset everything 
     o = 1; 
     i = 0; 
     cmd[0] = "\0"; 
     while (i < OPT_AMT) { 
      opt[i++] = "\0"; 
     } 

     // get input 
     printf("%s $ ", cwd); 
     scanf("%s", cmd); 
     gets(input); 
     opt[0] = cmd; 

     char *t = strtok(input, " "); 
     while (t != NULL) { 
      opt[o++] = t; 
      t = strtok(NULL, " "); 
     } 

     // if exit, exit 
     if (strcmp(cmd, "exit") == 0) { 
      exit = 1; 
     } 
     // else fork and execute 
     else { 
      pid_t pID = fork(); 

      if (pID == 0) { // child process 
       execvp(cmd, opt); 
      } else if (pID < 0) { // failed to fork 
       printf("\nFailed to fork\n"); 
      } else { // parent process 
       wait(0); 
      } 
     } 
    } 

    // cleanup 

    printf("\nFinished! Exiting...\n"); 
    return 0; 
} 

Все, что явно не так? Недавно я добавил условие выхода и сброс массива параметров.

Кроме того, это мой первый вопрос, поэтому напомните мне о любых правилах, которые я, возможно, сломал.

Answer 1

Для начала, это

cmd[0] = "\0"; 

должен быть

cmd[0] = '\0'; 

Слушайте предупреждения вашего компилятора.

Чтобы включить их, используйте опции -Wall -Wextra -pedantic (для gcc).

---

Кроме того, вы, возможно, лучше хотите initalise opt «s элементы не указывают на "ничего", то есть NULL но буквального "\0":

while (i < OPT_AMT) { 
     opt[i++] = NULL; 
    } 

, как execvp() требует opt быть NULL - (спасибо Paul за упоминание/формулировку соответствующего фона).

---

Также^2: Не используйте gets(), так как это зло, и даже не часть C стандарта больше. Вместо

gets(input); 

использования

fgets(input, sizeof input, stdin); 

gets() easyly давайте переливной пользователя по (вход) буфер передается. (Упоминание об этом пришло мне на ум без Paul, btw ... ;-))