我什么时候应该在C中使用free()



代码按照预期工作,尽管它从未释放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;
}

mallocfree视为"开始"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"释放内存。

相关内容

  • 没有找到相关文章

最新更新