實現一個簡單的shell

Question

使用fork和execvp函數從stdin行運行命令，製作一個簡單類型的shell。

但是，像ls這樣的工作，但不是ls -all -S。 它將執行ls，但什麼都不會被打印ls -all

唯一的想法，我能想出的是，有一個「\ n」某處的命令，但我不知道怎麼把它弄出來甚至它在哪裏......

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
#include <unistd.h> 
#include <time.h> 
#include <sys/types.h> 
#include <sys/wait.h> 
//Libs ^^^^ Defs vvvvvvvv 
#define comlen 4096      //Max command length 
#define comarg 32    //Max argument length 

int main(int argc, char *argv[]) 
{ 
    char buff; //command buffer 
    char* comand[comlen]; 
    int i; 
    do 
    { 
     i = 0; 
    printf("simsh: "); 

     char* whtspc = strtok (fgets(&buff, comlen, stdin)," "); //get input and tokenize 
     printf("[%lu] :: %s------------\nEND OF BUFF TEST\n", strlen(&buff), &buff); 

    while (whtspc != NULL) 
    { 
      comand[i]=(char*)malloc((sizeof(char)*strlen(whtspc))); //alloctie mem for commands 
      strncpy(comand[i], whtspc, strlen(whtspc)-1);      //coppy comand token to array index i 
      whtspc = strtok (NULL, " ");              //grab next token 
      i++;                          //incriment 
      /*trying to change new line character to NULL so that commands can be passed properly*/ 
//   if (comand[strlen(comand[i]) - 1] == "\n") 
//   { 
//    comand[strlen(comand[i]) - 1] = '\0'; 
//   } 
      //breka out incase index oversteps 
      if (i == 4096) 
       break; 
    } 
     //last entry in command should be null 
     comand[i] = NULL; 
     //fork to run in background 
     pid_t pid = fork(); 

     if (pid == 0) 
     { 
      //testing and pass comands to execvp 
      printf("START OF COMAND TEST\n!!!!!!!!!%s!!!!!!!!!!!!!!!!\n %lu\nEND OF COMAND TEST\n\n",comand[1], strlen(comand[0])); 
      execvp(comand[0], &comand); 
     } 

     else 
     { 
      //parent wait on child. 
      waitpid(pid, &i, WUNTRACED | WCONTINUED); 
     } 
    } 
    while(1); 

    return 0; 
} 

任何幫助將受到歡迎。

如果有幫助可言，這裏是代碼的終端輸出::

所有的

simsh: ls 
[3] :: ls 
------------ 
END OF BUFF TEST 
START OF COMAND TEST 
!!!!!!!!!(null)!!!!!!!!!!!!!!!! 
2 
END OF COMAND TEST 

chop_line.c chop_line.h list.c list.h Makefile Makefile~ One simsh1 simsh1.c simsh1.c~ 
simsh: ls -all 
[2] :: ls------------ 
END OF BUFF TEST 
START OF COMAND TEST 
!!!!!!!!!-all!!!!!!!!!!!!!!!! 
1 
END OF COMAND TEST 

simsh: echo 
[5] :: echo 
------------ 
END OF BUFF TEST 
START OF COMAND TEST 
!!!!!!!!!(null)!!!!!!!!!!!!!!!! 
4 
END OF COMAND TEST 


simsh: echo all 
[4] :: echo------------ 
END OF BUFF TEST 
START OF COMAND TEST 
!!!!!!!!!all!!!!!!!!!!!!!!!! 
3 
END OF COMAND TEST 

simsh: echo echo 
[4] :: echo------------ 
END OF BUFF TEST 
START OF COMAND TEST 
!!!!!!!!!echo!!!!!!!!!!!!!!!! 
3 
END OF COMAND TEST 

Answer 1

首先，你fgets正在讀一個單一字符buff。你應該讀入一個字符緩衝區。其次，fgets保持在讀字符串的結尾換行，所以你可能需要先刪除它，例如：

char buff[4096]; 
if (!fgets(buff, sizeof(buff), stdin)) { 
    // error or EOF 
    return 1; 
} 
int len = strlen(buff); 
if (len > 0 && buff[len-1] == '\n') { 
    buff[--len] = '\0'; 
} 
char *whtspc = strtok(buff, " "); 

還必須與buff替換所有引用&buff。

除此之外，您malloc也是錯誤的，並分配一個字符小於所需（strlen是沒有終止NUL）：

if (!(comand[i] = malloc(strlen(whtspc)+1))) { 
    return 1; // out of memory 
} 
(void) strcpy(comand[i], whtspc); 

相應的strncpy被複制比需要一個字符少。這是您的原始代碼意外地爲單字輸入工作的原因，因爲它在這種情況下會爲您刪除尾隨'\n'，但在所有其他情況下，它將刪除該單詞本身的最後一個字符。

，第二個參數，以execvp應該只是comand（原文如此）陣列：

execvp(comand[0], comand); 

Answer 2

的第一個參數fgets應的指針，其中的字符串被複制到緩衝區中。您正在傳遞一個指向單個的指針char。

其次，execvp需要兩個參數：一個文件名和一個空終止的命令行參數列表，按照慣例，它們以文件名本身開頭。

我冒昧地對您的代碼進行了一些修改，既解決了上面提到的問題，又使其更具可讀性。

請注意，下面的代碼中存在內存泄漏（修復它:)。可能還有其他問題，我沒有注意到。

前段時間我實現了一個shell;如果你想看看，我的GitHub網址在我的個人資料中（注意：醜陋的大學作業代碼）。

希望它有幫助！

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

#define COMLEN 4096                      
#define COMARG_N 32                      
#define TRUE 1 

int main(int argc, char *argv[]) 
{                          
    char *token;                       
    char *args[COMARG_N];                    
    char *buff;                       
    int i;                        
    pid_t pid;                       

    while(TRUE) {                      
     printf("simsh: ");                    

     buff = (char*) malloc(sizeof(char) * COMLEN);             
     fgets(buff, COMLEN, stdin);                 

     if (buff[strlen(buff) - 1] == '\n')               
      buff[strlen(buff) - 1] = '\0';                

     i = 0;                       
     token = strtok (buff, " ");                 

     while (token != NULL && i < COMARG_N - 1) {             
      args[i] = token;                   
      token = strtok (NULL, " ");                
      i++;                      
     }                        

     args[i] = NULL;                    

     pid = fork();                     
     if (pid == 0)                     
      execvp(args[0], &args[0]);                 
     else                       
      waitpid(pid, &i, WUNTRACED | WCONTINUED);             

     free(buff);                     
    }                         

    return 0;                        
}