从函数返回的空闲数组

Question

抱歉，如果这出现在其他地方，我无法找到明确的答案。 我一直在使用 Ed S 的答案，选项 1（链接如下）来分配内存，填充数组，然后将其返回给调用者。 他建议在你完成后释放内存，但是当我添加 free() 行时，我得到一个核心转储。 我曾在 GDB 上玩过，但我的技能可能不是我需要的。

提前感谢您提供的任何帮助。

回答链接：Returning an array using C

代码：

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

char * createArray();

int main(int argc, char *argv[]){
    printf("Creating Array...\n");
    // pointer to an int
    char *p;
    // size of the array
    int i,j;
    // width of array
    int width = 7;
    // height of array
    int height = 5;
    // get an array from the function
    p = createArray(width, height);
    // check if p was created properly
    if (p){
        // print out the array
        for (i = 0; i < width; ++i){
            for (j = 0; j < height; ++j){
                printf("[%c] ", *(p + (i * width) + j)); 
            }
        printf("\n");
        }

        // here's where it hits the fan
        free(p);
    }
    return 0;
}

char * createArray(int w, int h){
    // allocate some memory for the array
    char *r = malloc(w * h * sizeof(char));
    // check if the memory allocation was successful
    if(!r){
        return NULL;
    }
    int i, j;
    int count = 0;
    for (i = 0; i < w; ++i){
        for (j = 0; j < h; ++j){
            *(r + (i * w) + j) = 'X';
            ++count;
        }
    }
    return r;
}

Answer 1

有了这个

char *r = malloc(w * h * sizeof(char));

您分配了w * h（7 * 5 = 35 字节）的内存。但是

        *(r + (i * w) + j) = 'X';

可以访问远远超出您分配的 35 个字节（您将查看是否在循环中测试 i * w + j 的可能值），从而导致未定义的行为。

这可能会覆盖 malloc 的内部数据结构，因此您在 free() 时碰巧得到核心转储。

Answer 2

你在这些方面犯了一个错误

*(r + (i * w) + j) = 'X';

和

printf("[%c] ", *(p + (i * width) + j));

要保持在“2D”数组的边界内——它是一维的，但你正在像编译器一样处理它——它应该在那里是i * length：

*(r + (i * h) + j) = 'X';`

和

printf("[%c] ", *(p + (i * height) + j)); `

如果你使用它，你应该能够保持在边界内而不会弄得一团糟。