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

Question

代码按预期工作，但它从未释放由 malloc() 分配的内存。

我已经尝试在任何可能的地方释放内存，但无论我在哪里做，它都会破坏程序。具体来说，我收到“双重释放或损坏错误”。这更像是关于 free() 和 malloc() 实际做什么的问题？免费的所有问题主要是:

int main(int argc,  char *argv[]){
if(argc!=2){
    exit(1);
}

printf("CSA WC version 1.0\n\n");

int length = strlen(argv[argc-1]);
char file_to_open[length];
strcpy(file_to_open, argv[argc-1]);

//printf("filename:%s\n",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: %i\n",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: %s\n",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:%s\n",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] != '\0'){

    dest[i] = src[i];
    i++;

}

dest[i] = '\0';
//printf("length:%i\n",i);
free(dest);

return dest;
}

Answer 1

一般来说，您只需释放动态为您保留的内存。这意味着如果你有这样的声明:

int *my_int_pointer;
my_int_pointer = malloc(sizeof(int));

比您需要释放由 malloc 分配(保留)的内存。 如果您不确定在哪里释放它，而不是在程序结束时使用 free 释放它；

free(my_int_pointer);

在您的文件中，只要您读取的文件中有新行(在 while(done==0) 循环中)，就会分配内存。所以每次在 this 循环中的 if 之后，您都必须释放变量使用的内存。

此外，您需要释放为 readline 变量分配的内存。但正如之前指出的那样，您可能会在那里发生内存泄漏。

希望这对您有所帮助。

编辑:好的 - 我已经在想 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] != '\0'){
        dest[i] = src[i];
        i++;
    }
    dest[i] = '\0';
    //printf("length:%i\n",i);
    free(dest);                  /* This frees the memory, but you return a pointer */
    return dest;                 /* to this memory. this is invalid.                */
}

但是您可以释放的是该函数中的 src 指针。但请记住:在释放底层内存后，指针无法保存信息!它只是指向内存中不应再写入或读取的位置。

此外，只要没有'\0'，该函数就会复制字符串。如果没有终结者会怎样？该函数不断从一些不应该复制的内存地址复制!

你不应该使用那个函数;)