代码按照预期工作,尽管它从未释放malloc()
分配的内存。
我试着在任何可能的地方释放内存,但无论我在哪里做,都会破坏程序。具体来说,我得到了一个"双重免费或损坏错误"。这更像是一个关于free()
和malloc()
实际做什么的问题?免费的所有问题都是主要的:
int main(int argc, char *argv[]){
if(argc!=2){
exit(1);
}
printf("CSA WC version 1.0nn");
int length = strlen(argv[argc-1]);
char file_to_open[length];
strcpy(file_to_open, argv[argc-1]);
//printf("filename:%sn",file_to_open);
//create counters for output
int count_number_of_lines = 0;
int count_number_of_words = 0;
int count_number_of_characters = 0;
//create int size of default array size
int current_array_size = pre_read(file_to_open);
//printf("number of lines: %in",current_array_size);
//create string array of default size
char *strings_array[current_array_size];
//create a pointer to catch incoming strings
char *incoming_string=NULL;
int done=0;
while(done==0){
incoming_string=get_line_from_file(file_to_open, count_number_of_lines);
if(incoming_string!=NULL){
incoming_string=csestrcpy2(incoming_string);
//printf("incoming line: %sn",incoming_string);
strings_array[count_number_of_lines]=(char*)malloc(strlen(incoming_string+1));
strings_array[count_number_of_lines]=csestrcpy2(incoming_string);
//printf("added to array:%sn",strings_array[count_number_of_lines]);
count_number_of_lines++;
count_number_of_characters=(count_number_of_characters+(strlen(incoming_string)-1));
}
else{
done=1;
}
}
//all data is stored in a properly sized array
//count all words in array
int count=0;
int word_count=0;
char *readline;
while(count<current_array_size){
readline = csestrcpy2(strings_array[count]);
printf("line being checked: %s", readline);
int i=0;
int j=1;
while( j< strlen(readline)+1 ){
if(strcmp(readline,"n")!=0){
if( (readline[i] == ' ') && (readline[j] != ' ') ){
word_count++;
}
if( (readline[i] != ' ') && (readline[j] == 'n') ){
word_count++;
}
}
i++;
j++;
}
count++;
}
printf("current word count: %i", word_count);
return 0;
}
char* csestrcpy2(char* src){
int i = 0;
char *dest;
char t;
dest = (char*) malloc(MAX_LINE);
while( src[i] != ' '){
dest[i] = src[i];
i++;
}
dest[i] = ' ';
//printf("length:%in",i);
free(dest);
return dest;
}
通常情况下,您只需要释放为您动态保留的内存。这意味着如果你有这样的声明:
int *my_int_pointer;
my_int_pointer = malloc(sizeof(int));
而不是释放malloc分配(保留(的内存。如果你不确定在哪里释放它,而不仅仅是在程序结束时释放它,请使用free;
free(my_int_pointer);
在您的文件中,每当您读取的文件中有新行时(在while(done==0)
循环中(,似乎都会分配内存。所以每次在这个循环中的if
之后,你都必须释放变量使用的内存。
此外,您需要释放为readline变量分配的内存。但正如之前所指出的,你可能有内存泄漏。
希望这能有所帮助。
edit:好的,我已经在想csestrcpy
函数了。让我们看看这个功能:
char* csestrcpy2(char* src){
int i = 0;
char *dest;
char t;
dest = (char*) malloc(MAX_LINE); /*<<- This allocates memory that has to be freed*/
while( src[i] != ' '){
dest[i] = src[i];
i++;
}
dest[i] = ' ';
//printf("length:%in",i);
free(dest); /* This frees the memory, but you return a pointer */
return dest; /* to this memory. this is invalid. */
}
但是,您可以释放该函数中的src指针。但请记住:在底层内存释放后,指针无法保存信息!它只是指向记忆中一个不应该再写或读的地方。
此外,只要不存在"\0",函数就会复制字符串。如果没有终止符会发生什么?该函数不断从一些不该复制的内存地址复制!
您不应该使用该函数;(
每次成功调用malloc()
都需要调用free()
。
这并不一定意味着代码中需要相等数量的malloc()
和free()
调用;这意味着,对于程序运行时执行的每个malloc()
调用,都应该调用free()
,并将从malloc()
获得的指针值传递给它。CCD_ 14分配内存;free()
告诉系统您已经完成了分配的内存。
(几乎可以肯定的是,当程序终止时,您可以不使用free()ing
分配的内存,因为它将被操作系统回收,但出于风格和良好实践的考虑,您仍然应该将malloc()
s与free()
s相匹配。(
我正在忽略calloc()
和realloc()
呼叫。
动态内存分配(malloc(分配一个请求大小的内存块,并返回一个指向该块开头的指针。由于我们已经从内存中取出了这个块,所以在任务完成后将其返回内存是一个很好的做法。
现在作为您问题的答案,为了安全起见,您可以在返回之前调用free函数。
main{
int *i;
..
..
i=(int *)malloc(sizeof(int));
...//so something with i
..
free(i);
return 0;
}
将malloc
和free
视为"开始"one_answers"结束"。任何时候你打电话给malloc
,做你需要做的事,一旦你完成了,就永远打电话给free
。请确保只释放一次,两次释放是运行时错误。
如果你以某种方式丢失了malloc
返回的值(是的,这就是你的代码所发生的事情(,那么你就发生了内存泄漏(地狱之门打开了,亚达亚达(。
要重新迭代:释放malloc返回的任何内容(null除外(。
一般来说,每个malloc
都应该有一个相应的free
。然而,你不能free
东西两次(正如你现在已经注意到的那样(。我在你的代码中没有看到任何对free
的调用,所以不可能说出你的问题在哪里,但我马上注意到你在循环中malloc
有一些内存并将其分配给readline
,但在循环结束时你没有在readline
上调用free
,所以你在那里泄漏了内存。
此行:
strings_array[count_number_of_lines]=(char*)malloc(strlen(incoming_string+1));
正在被旁边的行覆盖,因此可以将其删除。
您还应该在最后一个循环中的count++
之后添加free(readline)
,以释放malloc创建的内存。
i++;
j++;
/// free here
free(readline);
}
count++;
}
printf("current word count: %i", word_count);
//free here as well
for(int k = 0; k < count_number_of_lines; k++)
{
free(strings_array[count_number_of_lines]);
}
return 0;
}
这应该行得通。
通常,在指针超出范围之前,使用calloc/malloc/realloc动态分配的任何内存都需要使用free((释放。
如果使用"new"分配内存,则需要使用"delete"释放内存。