0

我正在为命令行 shell 实现历史功能。我已经实现了一个循环数组来保存十个最近的命令。每个命令也由一个整数标记，指定哪个总命令是。例如，如果总共输入了 30 个命令，则循环数组中的 10 个命令将被编号（30、29、28、27、...、21）。

如果用户要插入命令“r”，后跟一个标记十条指令之一的数字，则该指令应该运行。在尝试确保正确接受两个字命令时，我一直遇到段错误。任何人都可以帮助指出问题所在。

int main(void)
{
    char inputBuffer[MAX_LINE]; /* buffer to hold the command entered */
    int background;             /* equals 1 if a command is followed by '&' */
    char *args[MAX_LINE/2+1];/* command line (of 80) has max of 40 arguments */

    int position, count, rnum = 0;
    char historyArray[10][MAX_LINE];
    char *holder[MAX_LINE]={0};

    while (1){            /* Program terminates normally inside setup */
        background = 0;
        printf("COMMAND->");
        fflush(0);

        setup(inputBuffer, args, &background);       /* get next command */

        position = (count % MOD_VAL);
        strcpy(historyArray[position],args[0]);

        if(!strcmp("rr",args[0]))
        {
            strcpy(historyArray[position],historyArray[((position-1)+MOD_VAL)%MOD_VAL]);
            printf("%i",count);
            printf("%c",'.');
            printf("%c",' ');
            printf("%s",historyArray[position]);
            printf("%c",'\n');
            strcpy(args[0],historyArray[position]);
        }

        else if(!strcmp("r",args[0])) //SEG FAULT OCCURING IN THIS ELSE-IF BLOCK!
        {
            //args[1] will hold given number
            printf("%c",'\n');
            printf("%s",args[0]);
            printf("%s",args[1]);
            printf("%s",args[2]);
            printf("%c",'\n'); //PRINT STATEMENTS FOR DEBUGGING

            strncpy(holder[0], args[2], MAX_LINE - 1); //SEG FAULT

            rnum = atoi(args[1]);
            strcpy(historyArray[position],historyArray[((position-(count-rnum))+MOD_VAL)%MOD_VAL]);
            strcpy(args[0],historyArray[position]); //CHANGES VALUES OF args[1], args[2]

            if(holder[0] != NULL)
            {
                strncpy(args[1],holder[0],MAX_LINE-1);
                args[2] = NULL;
            }
            else
            {
                args[1] = NULL;
            }

            printf("%c",'\n');
            printf("%s",args[0]);
            printf("%s",args[1]);
            printf("%s",args[2]);
            printf("%c",'\n');
        }

        else if(!(strcmp("h",args[0]))||!(strcmp("history",args[0])))
        {
            int counter = 0;
            while(counter < 10)
            {
                printf("%i",(count - counter));
                printf("%c",'.');
                printf("%c",' ');
                printf("%s", historyArray[((position - counter + MOD_VAL)%MOD_VAL)]);
                printf("%c",' ');
                printf("%c",'\n');
                counter ++;

                if(counter > count)
                    break;
            }
        }
        count++;

        pid_t pid1; //Initialize pid_t variable to hold process identifier
        pid1 = fork(); //Fork process and assign process identifier to "pid1"

        if (pid1 == 0) //Child process
        {
            //Child process executes the command specified by the user and
            //then quits.
            execvp(args[0], args);
            exit(0);
        }
        else //Parent process
        {
            if (background != 1)//Check for inclusion of '&' in command 
            {
                wait(NULL); //Wait for child process to finish executing
            }
        } 

        /* the steps are:
         (1) fork a child process using fork()
         (2) the child process will invoke execvp()
         (3) if background == 0, the parent will wait, 
         otherwise returns to the setup() function. */
    }
}

任何帮助表示赞赏！

-亚光

4

3 回答 3

3

这里你的 args 是字符指针数组。

但strcpy需要两个参数 - 应该是array或character pointer to which memory allocated by malloc

但是您 strcpy(historyArray[position],args[0]);认为一个论点character pointer不会被接受。

因此您可以将 更改args[]为args[][]或args[0] = malloc(some_no),segfault将被删除。

于 2013-03-08T04:07:02.310 回答
0

你注意到崩溃发生在线路上

else if(!strcmp("r",args[0]))

如果我是你，我会在调试器中加载核心文件并查看args[0]传递给strcmp().

char我希望您有关于和之间类型不匹配的编译器警告char*。您将 args 声明为char*. 这意味着args[0]是 a char，而不是 a char*。要比较单个字符，只需使用字符而不是strcmp()：

else if ('r' != args[0])

关于 C 字符串处理陷阱的一些注意事项：

  • strcmp()当其参数未正确以 NUL 终止时，对于数组边界是不安全的
    • 用于strncmp()限制比较的字符数
  • 尽管strncpy()可以防止数组边界，但它不能保证 NUL 终止目标字符串
  • strcpy()不尊重数组边界；您有责任确保目标数组足够大以接收复制到其中的字符串
于 2013-03-08T03:46:50.210 回答
-1

您缺少分配内存的char指针args和holder.

因此，通过函数族将它们称为指向以 0 结尾的字符数组（“字符串”）的指针str*()会导致未定义的行为，因为str*()函数试图取消引用那些不指向有效内存的指针。

于 2013-03-08T07:04:11.880 回答