我如何在这里释放内存？ （C）

Question

我有一个很快速的问题。当我尝试在void translateWord（）函数中取消分配数组时，为什么会检测到堆损坏？ 我试图在for循环中逐行取消分配，但它似乎不起作用。我认为如果我多次使用该功能，并且每次该功能分配内存时，我都应该在该功能结束时对其进行解除分配。有想法吗？

#define _CRT_SECURE_NO_WARNINGS
#include<stdio.h>
#include <conio.h>
#include <string.h>
#include <stdlib.h>
#include <ctype.h>


void translateWord(char word[], FILE *point_out, FILE *point_1)
{
    rewind(point_1);
    char ch;
    int gasit = 0;
    int lines = count_lines(point_1);
    int i = 0;
    char *cp;
    char *bp;
    char line[255];
    char **array = (char**)malloc(sizeof(char*)*2*lines);

    rewind(point_1);
    while (fgets(line, sizeof(line), point_1) != NULL) {
        bp = line;
        while (1) {
            cp = strtok(bp, "=\n");
            bp = NULL;

            if (cp == NULL)
                break;
            array[i] = (char*)malloc(sizeof(char)*strlen(cp));
            strcpy(array[i++], cp);
        }
    }

    gasit = cuvant_gasit(word, array, lines, point_out, gasit);
    if (gasit == 0)
    {
        fprintf(point_out, "<<%s>>", word);
    }

    for (int k = 0; k < 2 * lines; k++)
    {
        free(array[k]);
    }
    free(array);
}

Answer 1

translateWord 中有问题：

array[i] = (char*)malloc(sizeof(char)*strlen(cp));
strcpy(array[i++], cp);

第一行必须为array[i] = (char*)malloc(strlen(cp) + 1);，否则在 strcpy 期间缺少1个字符以保存最终的空字符。

请注意，根据定义，sizeof(char)是1。

为什么不只使用 strdup 而不是先使用 malloc 然后使用 strcpy ？只需将这两行替换为array[i++] = strdup(cp);