類Shell程序C

Question

下面的代碼應該可以像shell一樣工作。它具有上一個和下一個選項，歷史功能，退出和執行命令。

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
#include <unistd.h> 
#define BUFFER_SIZE 256 
#define HISTORY_LENGTH 128 

int executeCommand(char*cmd) 
{ 
    return(!strcmp(cmd,"e\n")); 
} 

int exitCommand(char*cmd) 
{ 
    return (!strcmp(cmd,"exit\n")); 
} 

int previousCommand(char*cmd) 
{ 
    return (!strcmp(cmd,"p\n")); 
} 

int nextCommand(char*cmd) 
{ 
    return (!strcmp(cmd,"n\n")); 
} 

void execute(char *line) 
{ 
    line[strlen(line)-1]='\0'; 
    char **arguments; 
    char* temp; 
    int i=0; 
    arguments=(char**)malloc(sizeof(char)*10); 
    temp=strtok(line," "); 
    arguments[i]=malloc(strlen(temp)*sizeof(char)); 
    if(arguments[i]!=NULL) 
    { 
     strcpy(arguments[i],temp); 
     i++; 
    } 
    else 
    { 
     printf("Out of memory"); 
    } 
    while(temp!=NULL) 
    { 
     temp=strtok(NULL," "); 
     if(temp==NULL){ 
      arguments[i]=NULL; 
     } 
     else{ 
      arguments[i]=malloc(strlen(temp)*sizeof(char)); 
      if(arguments[i]!=NULL) 
      { 
       strcpy(arguments[i],temp); 
       i++; 
      } 
     } 
    } 
    printf("%s ",arguments[0]); 
    printf("%s ",arguments[1]); 
    printf("%s ",arguments[2]); 
    execvp(arguments[0],arguments); 
} 

int main(int argc, char*argV[]) { 
    int i; 
    char *cmd=(char*)malloc(sizeof(char)*BUFFER_SIZE); 
    char **history=NULL; 
    int historylength=0; 
    int currentCommand=0; 
    history=(char**)malloc(sizeof(char)*BUFFER_SIZE); 
    do{ 
     fgets(cmd,BUFFER_SIZE-1,stdin); 
     if(exitCommand(cmd)) 
      break; 
     else 
      if(previousCommand(cmd)) 
      { 
       if(currentCommand>0) 
        printf("%s",history[--currentCommand]); 
       else if(currentCommand==0) 
       { 
        currentCommand=historylength; 
        printf("%s",history[--currentCommand]); 
       } 
      } 
      else 
       if(nextCommand(cmd)) 
       { 
        if(currentCommand<historylength) 
         printf("%s",history[currentCommand++]); 
       } 
       else 
        if(executeCommand(cmd)) 
        { 
         execute(history[--currentCommand]); 
        } 
        else 
        { 
         history[historylength]=malloc(strlen(cmd)*sizeof(char)); 
         if(history[historylength]!=NULL) 
         { 
          strcpy(history[historylength],cmd); 
          currentCommand=++historylength; 
         } 
         else 
         { 
          printf("Out of memory"); 
          break; 
         } 
        } 

    } while(1); 

    free(cmd); 

    for(i=0;i<historylength;i++) 
     free(history[i]); 
    free(history); 

    return 0; 
} 

我想使這個工作的功能貓。我輸入e cat main.c，我期望它執行cat命令，但它沒有做任何事情，我在這裏做錯了什麼？我不是專業人士，所以我感謝所有的幫助。

Answer 1

這是不正確的：

arguments=(char**)malloc(sizeof(char)*10); 

作爲arguments是char**，所以代碼需要分配sizeof(char*)。更改爲：

arguments = malloc(10 * sizeof(*arguments)); 

同樣錯誤爲history也。此外，請參閱Do I cast the result of malloc?

沒有比需要被分配爲char*爲strcpy()寫入終止空字符少一個char。變化：

arguments[i]=malloc(strlen(temp)*sizeof(char)); 

到：

arguments[i] = malloc(strlen(temp) + 1); 

sizeof(char)被保證是1並且可以從尺寸計算被省略。

防止i超過分配給arguments的內存的邊界。由於目前的代碼沒有防止i超過9（0到9是i的有效值，因爲arguments被分配到10元素）。