0

我应该为我的 char** 分配足够的内存。我使用gdb并找到了分段错误的点。我已经在这部分停留了大约一个小时，似乎无法弄清楚我为什么会出现段错误。

程序输出：

尺寸：10、20

开始： 1, 1

结束: 10, 20

分段错误（核心转储）

10 = m1.xsize
20 = m1.ysize
1 = m1.xstart
1 = m1.ystart
10 = m1.xend
20 = m1.yend

我的代码片段：

typedef struct mazeStruct
{
    char** arr;
    int xsize, ysize;
    int xstart, ystart;
    int xend, yend;
} maze;



/* read in the size, starting and ending positions in the maze */
    fscanf (src, "%d %d", &m1.xsize, &m1.ysize);
    fscanf (src, "%d %d", &m1.xstart, &m1.ystart);
    fscanf (src, "%d %d", &m1.xend, &m1.yend);

    /* print them out to verify the input */
    printf ("size: %d, %d\n", m1.xsize, m1.ysize);
    printf ("start: %d, %d\n", m1.xstart, m1.ystart);
    printf ("end: %d, %d\n\n", m1.xend, m1.yend);

    //allocating memory for 2d char array
    m1.arr = (char**)malloc(m1.xsize+2 * sizeof(char*));

    for(i = 0; i < m1.xsize+2; i++)
        m1.arr[i] = (char*)malloc(m1.ysize+2);

    /* initialize the maze to empty */
    for (i = 0; i < m1.xsize+2; i++) <---- when i = 6 it seg faults
        for (j = 0; j < m1.ysize+2; j++)
            m1.arr[i][j] = '.';

我没有分配足够的内存还是我做错了什么？

4

1 回答 1

5

你的表情：

m1.xsize + 2 * sizeof(char*)

相当于：

(m1.xsize) + (2 * sizeof(char*))

由于运算符的优先级，这不是您想要的。您需要改为使用：

(m1.xsize + 2) * sizeof(char*)

例如，假设您已m1.xsize设置为 20，并且您的指针大小为 4 个字节。因此，您需要 22 个指针的空间，即 88 个字节。该表达式m1.xsize + 2 * sizeof(char*)为您提供了 20 加上两倍的指针大小，总共 28 个字节，远远不足以满足您想要做的事情。

顺便说一句，您还应该停止转换返回值，malloc()因为它可以隐藏某些细微的错误。C 完全能够将void*返回的 from隐式转换malloc()为任何其他指针类型。

于 2013-03-06T06:36:31.960 回答