c-从矩阵中读取，用malloc分配，AddressSanitizer:堆缓冲区溢出

Question

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

#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

和像素.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

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

您对matr->grid本身的分配是奇怪的，尽管本身没有问题。您为struct pixel的w实例分配了足够的空间，但实际要存储的是指向struct pixel的w指针。只要struct pixel至少和struct pixel *一样大，分配的空间就足够大，但您确实应该通过分配确保足够大的空间来避免所有疑问，而且不会过大。

您对成员指针matr->grid所指向的空间的分配似乎是更严重的问题所在。对于每一个，您都分配了sizeof(matr->grid)*h字节，但实际上您似乎想要的是sizeof(struct pixel) * h字节。struct pixel很可能大于matr->grid(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 */

这里需要注意的事项：

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

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