使用 free 函数释放内存导致我的程序崩溃答案

【问题标题】：Freeing up memory with free function causing my program to crash使用 free 函数释放内存导致我的程序崩溃
【发布时间】：2017-05-25 09:11:36
【问题描述】：

我可以在我的程序中使用一些帮助，我写了一个程序来计算句子中字谜的数量，为此我使用了malloc()函数，你可以在我的代码**ArrPtr=malloc中看到。

我用它来计算字谜，完成后我想继续我的程序的第二部分，我想用free(arrPtr);释放内存并且程序崩溃（当我没有使用免费选项时它没有崩溃）。

这是我的代码，

void main()
{
    char str[1001] = { 0 };
    char temp[1001] = { 0 }, temp2;
    char strB[1001] = { 0 };
    int printf_i, counter, i, q, flag, j = 1, r = 0, m = 1, length = 0, root = 0, m1 = 0;
    int max_analogy = 0, counter2 = 0, O, sum, sum2;
    char **arrPtr;
    int k = 0;
    int **matrix;

    printf("Please enter the sentence, and then press Enter:\n");
    gets(str);

    //bubble sort
    strcpy_s(strB, 1001, str);
    for (i = 0; i < strlen(strB); i = q + 2)
    {
        do
        {
            flag = 0;
            for (q = i; strB[q + 1] != 32 && strB[q + 1] != 0; q++)
            {
                if (strB[q] > strB[q + 1])
                {
                    // Swap
                    temp2 = strB[q];
                    strB[q] = strB[q + 1];
                    strB[q + 1] = temp2;
                    flag = 1;
                }
            }
        } while (flag != 0);
    }
    counter = 1;
    length = strlen(strB);

    for (i = 0; strB[i] != 0; i++)
    {
        if (strB[i] == 32)
        {
            strB[i] = 0;
            counter++;
        }
    }

    arrPtr = (char*)malloc(sizeof(char)*counter);
    arrPtr[0] = strB;
    q = 1;
    for (i = 0; i < length - 1; i++)
    {
        if (strB[i] == 0)
        {
            arrPtr[q] = &strB[i + 1];
            q++;
        }
    }

    counter2 = 0;
    for (i = 0; i < counter; i++)
    {
        for (q = i + 1; q < counter; q++)
        {
            if (arrPtr[q] == 0 || arrPtr[i] == 0)
                continue;
            if (!strcmp(arrPtr[q], arrPtr[i]))
            {
                counter2++;
                arrPtr[q] = 0;
            }
        }
        if (max_analogy < counter2)
            max_analogy = counter2;
        counter2 = 0;
    }
    printf("The maximum number of anagram words in this sentence is %d.\n", max_analogy);

    free(arrPtr);
}

【问题讨论】：

你写的比分配给arrPtr的多。

标签： c memory malloc free

【解决方案1】：

arrPtr = (char*)malloc(sizeof(char)*counter);

错误的原因有很多：

arrPtr 是 (char **)。
使用 C 编译器进行强制转换既无用又危险。
你必须分配sizeof(char *)

原因 3 是您出现问题的真正原因：您在写入 counter*sizeof(char *)（很可能是 counter*8）时分配了 counter 字节，因此您正在写入超出分配内存的范围，从而破坏了 malloc 内存池.

你可以修复它

arrPtr = malloc(sizeof(char *)*counter);

【讨论】：