c - 从矩阵读取，用 malloc 分配，AddressSanitizer : heap-buffer-overflow

Question

所以我正在编写一个包含结构像素的矩阵。代码似乎将标准像素写入矩阵，但是当我尝试打印内容时，它似乎指向错误的地址，因为 AddressSanitizer 即将出现，printf 正在从错误的地址读取:以下是用于测试 printf() 分配的代码:

#include <stdio.h>
#include <stdlib.h>
#include "matrx.h"
#include "pixel.h"

void matr_initializer(struct matrix* matr, int w, int h){

matr->height = h;
matr->width = w;
struct pixel p;
standardPixel(&p);
matr->grid = (struct pixel**)malloc(sizeof(struct pixel)*w);

if(matr->grid == NULL){
   fprintf(stderr,"Irgendwas lief beim allozieren verkehrt");
   abort();
}

for(int i = 0; i < w; i++){
   matr->grid[i] = (struct pixel*)malloc(sizeof(matr->grid)*h);
}

for(int i = 0; i < w; i++){
   for(int j = 0; j < h; j++){
     matr->grid[i][j] = p;
   /*Here is the printf that causes the error*/
     printf("%d %d %d ",matr->grid[i][j].r,matr->grid[i][j].g,matr->grid[i][j].b);
}
   printf("\n");
}


 matr->n = w*h;
 matr->init = 1;

}

这是我正在使用的头文件:

 #ifndef _MATRH_
 #define _MATRH_
 #include <stdio.h>
 #include <stdlib.h>
 #include "pixel.h"
 // typedef struct matrix matrix;

 struct matrix{
 int height;
 int width;
 struct pixel*  spalten;
 struct pixel** grid;
 int n;
 int init;
 };

 void matr_initializer(struct matrix* matr, int w, int h);


 void printf_matr_color(struct matrix* matr);

 void printf_matr_RGB(struct matrix* matr);
 #endif

还有pixel.h

#ifndef _PIXELH_
#define _PIXELH_
#include <stdio.h>
#include <stdlib.h>

struct pixel{
  int color;
  int r,g,b;
  int brightness;
  int energy;
};

void standardPixel(struct pixel* p);
#endif

Answer 1

成员(member)grid的struct matrix被声明为 struct pixel ** ，并且您似乎有意将其用作指向动态分配数组的指针的动态分配数组。这很好。

您对 matr->grid 的分配它本身很奇怪，尽管本身并没有问题。您分配足够的空间 w struct pixel 的实例，但您实际打算存储的内容有 w 指向 struct pixel 的指针 。分配的空间足够大只要struct pixel至少与struct pixel *一样大，但您确实应该通过分配保证足够大小的空间来避免所有疑问，而且不要过多。

您对成员指针指向的空间的分配matr->grid这才是更严重的问题所在。对于您分配的每一个 sizeof(matr->grid)*h字节，但您似乎真正想要的是 sizeof(struct pixel) * h字节。很可能是struct pixel大于matr->grid (a struct pixel ** )，在这种情况下，您没有分配所需的内存。

这似乎是您真正想要的:

matr->grid = malloc(sizeof(*matr->grid) * w);
for(int i = 0; i < w; i++){
   matr->grid[i] = malloc(sizeof(*matr->grid[i]) * h);
}
/* error checking omitted for brevity */

这里需要注意的事项:

• 没有必要，通常也不希望强制转换 malloc() 的返回值在C中
• sizeof运算符不计算其操作数；它仅使用操作数的类型(有一个异常(exception)，此处不适用)。
• 因此，根据指针引用的大小来计算要分配的字节数是有效的，如所示。这可以确保您使用正确的元素大小，即使您更改指针的类型也是如此。

此外，请注意，尽管您的索引似乎与您的分配和尺寸标注一致，但按照您的方式，您的网格将按 [column][row] 进行索引。 。更典型的安排是按 [row][column] 进行索引。 ，相反。