2 
#pragma pack(push, 1)
typedef struct 
{
      /*...*/
  unsigned int      dataoffset;         //No of bytes before actual pixel data
}HEADER;

typedef struct 
{
   /*...*/
   unsigned int     width;
   unsigned int     height;
   unsigned short   bits_per_pixel;         //code is written for 24 bits only and No other format is supported..
    /*...*/
}INFO_HEADER;

typedef struct 
{
   unsigned char    b;
   unsigned char    g;
   unsigned char    r;        
}COLORMAP;

#pragma pack(pop)


int main()
{
      // Var decl.
  INFO_HEADER       *pHeader = NULL;
  FILE              *pImage;
  COLORMAP          **ppColors;
  /*...*/

/* File opened in read, binary mode, memory allocated for pHeader*/     

fread (pHeader, sizeof(INFO_HEADER), 1, pImage);  

/*Next block is actually problematic.. Posting 'as is' from my code*/
      ppColors = (COLORMAP**)malloc((pHeader -> height ) * sizeof(COLORMAP));
  for(i = 0 ; i < pHeader -> height ; i++)
     ppColors[i] = (COLORMAP*)malloc(pHeader -> width * sizeof(COLORMAP));
fseek(pImage, pHeader -> fileheader.dataoffset, SEEK_SET);
    for (i = 0 ; i < pHeader -> width ; i++) 
  {
      for (j = 0 ; j < pHeader -> height ; j++) 
      {
          fread(&b, sizeof(unsigned char), 1, pImage);
          fread(&g, sizeof(unsigned char), 1, pImage);
          fread(&r, sizeof(unsigned char), 1, pImage);

          ppColors[i][j].b = b;
          ppColors[i][j].g = g;
          ppColors[i][j].r = r; 

          printf("width = %d height = %d %d:\t", i, j,  cnt);
          printf("%d ", (int)ppColors[i][j].b);
          printf("%d ", (int)ppColors[i][j].g);
          printf("%d\n", (int)ppColors[i][j].r);
          cnt++;  
      }
  }

  /*And at last free()ing..*/
  for(i = 0 ; i < pHeader -> height ; i++)        free(ppColors[i]);
   free(ppColors);
  cleanup();
  return(0)
}

可能重复：http ://stackoverflow.com/questions/1568042/optimal-way-to-free-a-malloced-2d-array-in-c

虽然上面的链接无法解决我的问题。

  1. 我的内存用完了。我对高度进行了 malloc() 处理，然后对于每个高度，宽度再次进行了 malloc() 处理。我正在尝试仅在宽度 X 高度域上工作。看来问题出在身高上。如果你改变

ppColors = (COLORMAP**)malloc((pHeader -> height ) * sizeof(COLORMAP));至 ppColors = (COLORMAP**)malloc((pHeader -> height + 6 ) * sizeof(COLORMAP));

然后这个问题就消失了。

  1. 但是在 free()ing 时，我在核心转储中得到双重释放/损坏。

我该死的肯定我在某个地方出错了。我不希望有人纠正我的代码，我只是运行它。只是提示就可以了。

4

3 回答 3

3

指向指针的指针与数组数组不同。

当使用指针来模拟多维数组时，像这样声明：

char **pointer;

内存看起来像这样：

+------------+------------+------+------ --+
| 指针[0] | 指针[1] | ... | 指针[大小 - 1] |
+------------+------------+------+------ --+
     | | |
     vvv
   数据 数据 数据

而一个合适的多维数组

char array[X][Y];

在内存中看起来像这样：

+-------------+-------------+------+--------------- --+-------------+------+
| 数组[0][0] | 数组[0][1] | ... | 数组[0][Y - 1] | 数组[1][0] | ... |
+-------------+-------------+------+--------------- --+-------------+------+

因此，在一个适当的多维数组中，所有内存都在一个块中，而使用指针到指针时，您有一个指针数组而不是数组数组。

所以你应该做的是分别分配所有子数组：

ppColors = malloc(pHeader->height * sizeof(COLORMAP *));
/* Note how I allocate the width times the size of a COLORMAP pointer */

for (int i = 0; i < pHeader->height; i++)
    ppColors[i] = malloc(pHeader->width * sizeof(COLORMAP));

不要忘记您现在也必须free循环调用！

于 2013-01-17T13:12:47.237 回答
2

我可以看到几个问题：

  • ppColors是一个指针数组。数组中的每个元素都是 a COLORMAP*，因此您需要使用 计算要分配的大小numElements * sizeof(COLORMAP*)。 COLORMAP只有 3 个字符，所以很可能sizeof(COLORMAP*)> sizeof(COLORMAP)。您当前的分配将太小，因此您最终会写到数组末尾之外；这具有未定义的影响，但可能会崩溃。
  • 宽度和高度的使用在分配和循环之间是相反的，因此您最终会在循环中的某个点写入未分配的内存。
于 2013-01-17T13:11:48.630 回答
1

This may not be an answer but probably help you along with other answers.

  • This might be problem with fread check what do you get in pHeader -> height and pHeader -> width

  • Use parenthesis around pHeader -> width in malloc(pHeader -> width * sizeof(COLORMAP))

于 2013-01-17T13:17:07.130 回答