我正在自学C。现在我正在努力制作一个外壳，部分基于
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 != ' ')
{
if(*thestring!=' '){
returnline[charpos]=*thestring;
charpos++;
}
thestring++;
}
returnline[charpos]=' ';
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("%sn",linenospace);
token=strtok(linenospace,"|");
//printf("%s first tokenn",token);
int argsub=0;
while (token!=NULL)
{
returnargs[argsub]=token;
//printf("%sn",token); //test that all arguments are read
argamounts++;
token=strtok(NULL,"|");
//printf("%s second tokenn",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 returnarg0n", returnargs[0]);
//printf("%s returnarg1n", returnargs[1]);
return returnargs;
}
但是 shell 没有读取参数，并且在插入所有 printf 以找出参数落入的位置时，我意识到释放"linenospace"会丢弃我的参数。因此，如果 strtok 返回一个设置为 token 的指针，并且返回参数的"元素"是令牌指针，那么释放"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并检查了内存泄漏。果然有一个，并将"free(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 会不断循环回来，如果我继续错误定位，我需要一遍又一遍地释放。
但是，出于某种原因，如果我遍历所有参数并尝试释放它们，我会从 valgrind 得到"无效的 free()"。我必须释放参数[0]或内存泄漏，但我只能释放参数[0]，不能再释放了。
添加：
printf("amount of args %in",argamounts);
int freedouble;
for(freedouble=0; freedouble<argamounts; freedouble++)
{
printf("argument %d is %s ",freedouble,args[freedouble]);
//free(args[freedouble]);
}
进入 do while 循环以检查是否所有参数都已注册，表明它们都是，但我无法一一释放它们。一旦我弄清楚原因，我将再次编辑它，但如果有人知道，请告诉我。
但是，出于某种原因，如果我遍历所有参数并尝试释放它们，我会从 valgrind 得到"无效的 free()"。我必须释放参数[0]或内存泄漏，但我只能释放参数[0]，不能再释放了。
你不能释放args[1]等，因为你没有错误地放置它们。关于args[0]，你也没有完全错误地分配它，但args[0]指向linenospace=nospace(line)分配的内存空间中的第一个标记，通常在它的开头(除非该行以|开头)，因此你大多可以滥用args[0]来释放nospace(line)分配的内存。
但是，nospace(line)是无用的，因为删除了所有空格的命令，即所有参数连接在一起，是无法识别的(除非没有参数)。因此，我建议从程序中完全删除nospace();这样也就不用担心额外的内存分配了。