0

我正在尝试取消分配使用以下代码分配的第二个数组：

int** createMatrix(int k)
{
    int i;
    int **res;

    res = (int**)malloc(sizeof(int*)*k);
    checkalloc(res);

    for (i = 0; i < k; i++)
    {
        res[i] = (int*)malloc(sizeof(int)*k);
        checkalloc(res[i]);
    }
    return res;
}

具有以下功能

void freeMatrix(int ***matrix, int k)
{
    int i;

    for (i = 0; i < k; i++)
        free(matrix[i]);

    free(matrix);
}

第一个元素被很好地释放，但是当i=1函数因错误而崩溃时：

First-chance exception at 0x0f7e7e2c (msvcr100d.dll) in Q2.exe: 0xC0000005: Access violation reading location 0xccccccc8.
Unhandled exception at 0x0f7e7e2c (msvcr100d.dll) in Q2.exe: 0xC0000005: Access violation reading location 0xccccccc8.

我究竟做错了什么？

编辑：

尝试解除分配由以下人员分配的类似数组时，我也会发生同样的情况：

CHARS** createImageMatrix(FILE *fp, int rows, int cols, int format)
{
    CHARS**res;
    int i;

    res = (CHARS**)malloc(sizeof(CHARS*)*rows);
    checkalloc(res);

    for (i = 0; i < height; i++)
    {
        res[i] = (CHARS*)malloc(sizeof(CHARS)*cols);
        checkalloc(res[i]);
    }
    return res;
}

并解除分配：

void freeCharsMatrix(CHARS **matrix, int rows)
{
    int i;
    for (i = 0; i < rows; i++)
        free(matrix[i]);
    free(matrix);
}

当 CHARS 是：

typedef struct Chars {
char A;
char B;
char C;
char D;

} 字符；

但这次我得到：

错误

我怎么了？

4

2 回答 2

4 
void freeMatrix(int ***matrix, int k)

==>

void freeMatrix(int **matrix, int k)

编辑：

free(matrix+i) // when you do this the pointer across the whole array .

==>

free(matrix[i])
于 2013-08-21T21:25:18.240 回答
1

您的“ImageMatrix”示例有两个主要问题：

  1. 您的createImageMatrix()函数不返回值。
  2. 您的freeCharsMatrix()函数free(matrix+i)在应该使用free(matrix[i])or时调用free(*(matrix+i))。

您的附属问题与编译相关：

  1. createImageMatrix()不使用fporformat参数。
  2. 函数的主体是用 和 编写的height，width但参数是rows和cols。

这是您的代码的调试版本。我已重命名为freeCharsMatrix()，freeImageMatrix()因此名称更加一致。当我进行主要测试时，func_name引用拼写为__func__，但这是 C99 功能（GCC 4.8.1 甚至允许使用-std=c89 -pedantic），因此我重命名了变量并提供了显式定义，因为您很可能使用 C89 编译器（ MSVC 只支持 23 年的标准，不支持 13 年的标准或 1 年的标准）。

你可以看到我是如何最终诊断问题的——打印出传递给的地址free()并将它们与malloc(). 我花了很长时间才发现这matrix[i]相当于*(matrix+i)而不是matrix+i。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

typedef struct Chars
{
    char A;
    char B;
    char C;
    char D;
} CHARS;

void dumpImageMatrix(CHARS **matrix, int rows, int cols);
CHARS **createImageMatrix(int height, int width);
void freeImageMatrix(CHARS **matrix, int height);

static void dump_address(const char *tag, int num, void *vp)
{
    printf("%s[%d] = %p\n", tag, num, vp);
}

static void checkalloc(void *vp)
{
    if (vp == 0)
    {
        fprintf(stderr, "Memory allocation failed\n");
        exit(1);
    }
}

CHARS **createImageMatrix(int height, int width)
{
    static const char func_name[] = "createImageMatrix";
    CHARS **res;
    int i;

    printf("-->> %s()\n", func_name);
    res = (CHARS**)malloc(sizeof(CHARS*)*height);
    checkalloc(res);
    dump_address("MTX", 0, res);

    for (i = 0; i < height; i++)
    {
        int j;
        res[i] = (CHARS*)malloc(sizeof(CHARS)*width);
        checkalloc(res[i]);
        dump_address("Row", i, res[i]);
        for (j = 0; j < width; j++)
            memset(&res[i][j], (i + j) % 26 + 'A', sizeof(CHARS));
    }
    printf("<<-- %s()\n", func_name);
    return res;
}

void freeImageMatrix(CHARS **matrix, int height)
{
    static const char func_name[] = "freeImageMatrix";
    int i;
    printf("-->> %s()\n", func_name);
    for (i = 0; i < height; i++)
    {
        dump_address("Row", i, matrix[i]);
        /*dump_address("CHK", i, matrix+i);*/
        /*dump_address("CHK", i, *(matrix+i));*/
        /*free(matrix[i]);*/
        free(*(matrix+i));
        /*free(matrix+i);*/
    }
    dump_address("MTX", 0, matrix);
    free(matrix);
    printf("<<-- %s()\n", func_name);
}

int main(void)
{
    enum { m_rows = 5, m_cols = 6 };
    CHARS **mat = createImageMatrix(m_rows, m_cols);
    dumpImageMatrix(mat, m_rows, m_cols);
    freeImageMatrix(mat, m_rows);
    return 0;
}

static void dump_Chars(CHARS c)
{
    printf("%c%c%c%c", c.A, c.B, c.C, c.D);
}

void dumpImageMatrix(CHARS **matrix, int rows, int cols)
{
    static const char func_name[] = "dumpImageMatrix";
    int i;
    printf("-->> %s()\n", func_name);
    dump_address("MTX", 0, matrix);
    for (i = 0; i < rows; i++)
    {
        int j;
        dump_address("Row", i, matrix[i]);
        for (j = 0; j < cols; j++)
        {
            if (j != 0)
                putchar(' ');
            dump_Chars(matrix[i][j]);
        }
        putchar('\n');
    }
    printf("<<-- %s()\n", func_name);
}

样本输出：

-->> createImageMatrix()
MTX[0] = 0x7f97cb4039d0
Row[0] = 0x7f97cb403a00
Row[1] = 0x7f97cb403a20
Row[2] = 0x7f97cb403a40
Row[3] = 0x7f97cb403a60
Row[4] = 0x7f97cb403a80
<<-- createImageMatrix()
-->> dumpImageMatrix()
MTX[0] = 0x7f97cb4039d0
Row[0] = 0x7f97cb403a00
AAAA BBBB CCCC DDDD EEEE FFFF
Row[1] = 0x7f97cb403a20
BBBB CCCC DDDD EEEE FFFF GGGG
Row[2] = 0x7f97cb403a40
CCCC DDDD EEEE FFFF GGGG HHHH
Row[3] = 0x7f97cb403a60
DDDD EEEE FFFF GGGG HHHH IIII
Row[4] = 0x7f97cb403a80
EEEE FFFF GGGG HHHH IIII JJJJ
<<-- dumpImageMatrix()
-->> freeImageMatrix()
Row[0] = 0x7f97cb403a00
Row[1] = 0x7f97cb403a20
Row[2] = 0x7f97cb403a40
Row[3] = 0x7f97cb403a60
Row[4] = 0x7f97cb403a80
MTX[0] = 0x7f97cb4039d0
<<-- freeImageMatrix()
于 2013-08-22T02:54:43.220 回答