Question

我正在教自己C.现在我正在制作一个shell，部分基于

https://brennan.io/2015/01/16/write-a-shell-in-c/

我正在尝试添加bash中的管道并创建一个名为“nospace”的函数来消除参数之间的空格，以便strtok将基于“|”分隔。

char* nospace(char *thestring)
{
char* returnline=(char*)malloc(sizeof(char)*50);
int charpos;
charpos=0;

while(*thestring != '\0')
{
    if(*thestring!=' '){
        returnline[charpos]=*thestring;
        charpos++;
    }


    thestring++;
}

returnline[charpos]='\0';


return returnline;
}

因为我使用malloc作为返回行，所以我正在读取SO，我需要将它释放到某处，所以自read_args调用nospace后我在read_args中释放它。

char** read_args(char* line)
{
argamounts=0;
//tokens and strtok was taken from tutorialspoint regarding the strtok    function
char**returnargs = (char**) malloc(sizeof(char*)*20);
char* token;

char* linenospace=nospace(line);

//printf("%s\n",linenospace);


token=strtok(linenospace,"|");
//printf("%s first token\n",token);
int argsub=0;

while (token!=NULL)
{
    returnargs[argsub]=token;
    //printf("%s\n",token); //test that all arguments are read
    argamounts++;
    token=strtok(NULL,"|");
    //printf("%s second token\n",token);
    argsub++;
}

//printf("%d",argamounts);
//returnargs[0]=line; //assumes only one arg for now

//cannot free memory here or returnargs is null, why?
//free(linenospace);

//printf("%s returnarg0\n", returnargs[0]);
//printf("%s returnarg1\n", returnargs[1]);
return returnargs;
}

但是shell没有读取参数，并且在插入所有printf以找出参数落在哪里后，我意识到释放“亚麻布”会使我的论点失败。因此，如果strtok返回一个设置为token的指针，并且返回args的“元素”是令牌指针，那么“linenospace”被释放的方式是否必须释放shell循环函数中的双指针？

void ypsh_loop(void)
{
  char *line;
  char **args;
  int status;

  do {
  printf("ypsh > ");
  line = read_line();
  args=read_args(line);
  status=shexecute(args);

  }while(status);

  free(line);
  free(args);
}

（我想我必须更改free（args）;行以释放双指针）。

实际上在写这个问题的过程中，我通过SO（CentOS是我的家庭操作系统）快速搜索后下载了valgrind并检查了内存泄漏。果然有一个并改变了“自由（args）;”到

int freedouble;
for(freedouble=0; freedouble<argamounts; freedouble++)
{
    free(args[freedouble]);
}

free(args);

其中argamounts是一个全局管理的变量，似乎解决了这个问题。我想这会回答我的问题，但无论如何我都会在这里发布。

编辑：

显然循环函数需要以这种方式编写：

void ypsh_loop(void)
{
  char *line;
  char **args;
  int status;

  do {
  printf("ypsh > ");
  line = read_line();
  args=read_args(line);
  status=shexecute(args);

     free(line);
     free(args[0]);
     free(args);

  }while(status);


}

将free（）语句移动到do while循环中，而不是之前的那些语句是有意义的，因为shell保持循环，如果我保持malloc-ing，我需要一遍又一遍地释放。

但是，出于某种原因，如果我遍历所有的args并尝试释放它们，我会从valgrind获得“Invalid free（）”。我必须释放args [0]或内存泄漏，但我只能释放args [0]而不能更新。

添加：

    printf("amount of args %i\n",argamounts);
    int freedouble;
    for(freedouble=0; freedouble<argamounts; freedouble++)
    {
        printf("argument %d is %s ",freedouble,args[freedouble]);
        //free(args[freedouble]);
    }

进入do while循环以检查所有参数是否已注册表明它们都是，但我不能逐个释放它们。一旦我找出原因，我将再次编辑这个，但如果有人知道，请告诉我。

Answer 1

但是，由于某种原因，如果我遍历所有args并尝试释放它们，我会从valgrind中获得“无效的free（）”。我必须释放args [0]或内存泄漏，但是我只能释放args [0]，不能再释放。

您无法释放args[1]等，因为尚未分配它们。关于args[0]，您也没有完全对其进行分配，但是args[0]指向linenospace=nospace(line)所分配的内存空间中的第一个标记，通常是在其开头（除非该行开始|），因此您通常可以滥用args[0]来释放nospace(line)分配的内存。

但是，nospace(line)是无用的，因为删除了所有空格的命令，即i。 e。所有连接的参数都是无法识别的（除非没有参数）。因此，我建议从程序中完全删除nospace()；这样就不用担心额外的内存分配了。

意外的free（）行为

1 个答案: