2

我有一个基本程序,它通过 readline 函数从用户那里接收用户命令“command varname = variable” 。然后它解析字符串以将每个部分存储到字符串变量中。我遇到的问题是我收到一个错误:分段默认值(核心转储),它应该来自非法内存访问,但是我已经在示例输入中逐行浏览了我的程序:“设置ray”,我不知道这是在哪里发生的。我的预期结果是Command 应该包含 set,Varname 应该包含 ray,Value 应该是 NULL 这是代码以及显示错误的示例输入和输出。我还看到了一个错误:bus (core dumped)出现一次。这个错误来自哪里的任何想法?

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <string.h>
#include <readline/readline.h>
#include <readline/history.h>

/* Simple example of using gnu readline to get lines of input from a user.
   Needs to be linked with -lreadline -lcurses
   add_history tells the readline library to add the line to it's
   internal histiry, so that using up-arrow (or ^p) will allows the user
   to see/edit previous lines.
*/


int main(int argc, char **argv) {

  char * s;
  char * Command;
  char * Varname;
  char * Value;

  while (s=readline("Enter Name: ")) {
    add_history(s); /* adds the line to the readline history buffer */

    printf("Hello %s\n",s);/*output message to the user*/

    int part = 1;
    int i;

    for (i = 0; i < strlen(s); i++)
      {

        while((isspace(s[i]) || s[i] == '=') && (i < strlen(s)))
          {
            i++;//parse the string for the next portion of the command            
          }

        if(i >= strlen(s))
          {
            printf("Error: Command not properly formatted!\n");
            break;//terminate the for loop
          }        

        if(part == 1)//grab the command
          {
            int size1 = 0;//size of the command
            int j = i;//index of the first non-space character in this portion of the string

            while(!isspace(s[j]))//determine the size of the command, j will point to space when the loop exits
              {
                size1++;
                j++;                            
              }

            Command = (char*) malloc (size1+1);//allocate space to hold the characters of the command along with the terminating '\0' character

            int d = 0;//destination index
            while(i<j)
              {
                Command[d] = s[i];//copy the characters of the command portion of s over to the command array
                d++;
                i++;
              }

            Command[d] = '\0';//terminating null character
            part++;//increment the part of the command

          }
        else if(part == 2)//grab the varname
          {
            int size2 = 0;//initialize the size of the character array which will hold the variable name
            int k = i;
            while(!isspace(s[k]))//determine the size of the varname, k will be at the next space when the loop exits
              {
                size2++;
                k++;                            
              }

            Varname = (char*) malloc (size2+1);//allocate space to hold the characters of the Varname along with the terminating '\0' character

            int e = 0;
            while(i<k)
              {
                Varname[e] = s[i];//copy the characters of the Varname portion of s over to the Varname string
                e++;
                i++;
              }
            Varname[e] = '\0';
            part++;//increment part
          }
        else if(part == 3)//grab the value if one is given
          {
            int size3 = 0;//initialize the size of the character array which will hold the value of the environment variable
            int l = i;
            while(!isspace(s[l]) && s[l] != '\0')//determine the size of the varname and check if we've reached the end of the string
              {
                size3++;
                l++;                            
              }

            Value = (char*) malloc (size3+1);//allocate space to hold the characters of the Value along with the terminating '\0' character
            int f = 0;
            while(i<l)
              {
                Value[f] = s[i];//copy the characters of the command portion of s over to the command array
                f++;
                i++;
              }
            Value[f] = '\0';
          }
      }
    /*print out the sections of the full command*/
    if(Command != NULL)
      {
        printf("Hey %s\n",Command);
      }
    if(Varname != NULL)
      {
        printf("Hey %s\n",Varname);
      }  
    if(Value != NULL)
      {
        printf("Hey %s\n",Value);
      }

    /* clean up! */
    free(s);
    free(Command);
    free(Varname);
    free(Value);
  }
  return(0);
}

样本输入/输出:

Enter Name: set ray = 21
Hello set ray = 21
Hey set
Hey ray
Hey 21
Enter Name: print ray
Hello print ray
Hey print
Hey ray
Hey
Enter Name: delete varname
Hello delete varname
Hey delete
Hey varname
Hey delete
Enter Name: print raymond
Hello print raymond
Hey print
Hey raymond
Hey
Enter Name: delete variable
Hello delete variable
Hey delete
Hey variable
Segmentation fault (core dumped)
4

1 回答 1

1

您声明三个指针而不初始化它们

char * Command;
char * Varname;
char * Value;

根据用户输入的内容,这些可能不会全部初始化为指向for循环内动态分配的内存。如果它们没有全部初始化,free稍后尝试它们可能会崩溃

free(Command);
free(Varname);
free(Value);

最简单的解决方法是NULL在开始处理每一行时初始化所有变量

while (s=readline("Enter Name: ")) {
    Command = NULL;
    Varname = NULL;
    Value = NULL;

这样就可以安全地调用free它们,即使它们仍设置为NULL.

于 2013-10-12T21:02:03.607 回答