Question

我正在为命令行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. */
    }
}

感谢任何帮助！

-MATT

Answer 1

这里你的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将被删除。

Answer 2

您注意到崩溃发生在

行

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

如果我是你，我会在调试器中加载核心文件，看看传递给args[0]时strcmp()的值是多少。

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

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

关于C字符串处理的陷阱的一些注释：

strcmp()在数组边界的参数未正确NUL终止时是不安全的
- 使用strncmp()来限制比较的字符数
虽然strncpy()防范数组边界，但不保证NUL终止目标字符串
strcpy()不尊重数组边界;您有责任确保目标数组足够大以接收要复制到其中的字符串

Answer 3

您错过了为char和args中保留的holder指针分配的内存。

因此，通过str*()函数族引用那些作为0终止字符数组（“字符串”）的指针，导致未定义的行为，因为str*()函数试图降低那些指针而不是指向有效的记忆。

分段错误 - strcpy（） - C.

3 个答案: