3

我试图从 24 位 BMP 文件中获取 RGB 值。我使用的图像是一个很小的图像，全是红色的，所以所有像素 BGR 配置应该是B:0 G:0 R:255. 我这样做：

int main(int argc, char **argv)
{
    principal();
    return 0;
}

typedef struct {
    unsigned char blue;
    unsigned char green;
    unsigned char red;
} rgb;

typedef struct {
    int ancho, alto;
    rgb *pixeles[MAX_COORD][MAX_COORD];
} tBitmapData;

void principal()
{

    FILE *fichero;
    tBitmapData *bmpdata = (tBitmapData *) malloc(sizeof(tBitmapData));
    rgb *pixel;
    int i, j, num_bytes;
    unsigned char *buffer_imag;
    char nombre[] = "imagen.bmp";
    fichero = fopen(nombre, "r");
    if (fichero == NULL)
            puts("No encontrado\n");
    else {
            fseek(fichero, 18, SEEK_SET);
            fread(&(bmpdata->ancho), sizeof((bmpdata->ancho)), 4, fichero);
            printf("Ancho: %d\n", bmpdata->ancho);
            fseek(fichero, 22, SEEK_SET);
            fread(&(bmpdata->alto), sizeof((bmpdata->alto)), 4, fichero);
            printf("Alto: %d\n", bmpdata->alto);
    }

    num_bytes = (bmpdata->alto * bmpdata->ancho * 3);
    fseek(fichero, 54, SEEK_SET);
    for (j = 0; j < bmpdata->alto; j++) {
            printf("R   G   B Fila %d\n", j + 1);
            for (i = 0; i < bmpdata->ancho; i++) {
                    pixel =
                        (rgb *) malloc(sizeof(rgb) * bmpdata->alto *
                                       bmpdata->ancho * 3);
                    fread(pixel, 1, sizeof(rgb), fichero);
                    printf("Pixel %d: B: %3d G: %d R: %d \n", i + 1,
                           pixel->blue, pixel->green, pixel->red);
            }
    }
    fclose(fichero);
}

问题是当我打印它们时，第一个像素很好，B:0 G:0 R:255但随后它们开始变为B:0 G:255 R:0，然后变为B:255 G:0 R:0。如果宽度为 10 像素，则每 10 像素发生一次更改。

4

2 回答 2

5

在BMP 文件格式中，可以填充每一行像素数据，以便向上舍入为 4 字节的倍数。

如果您有 10 个 24 位像素，则为 30 个字节，然后是 2 个字节的填充。您的代码不会跳过填充。

于 2011-03-19T00:39:25.693 回答
2

我认为你的fread(3)电话是错误的：

        fread(&(bmpdata->ancho), sizeof((bmpdata->ancho)), 4, fichero);

这要求将4*sizeof((bmpdata->ancho))字节读入int. 我假设sizeof((bmpdata->ancho))return 4，所以我认为你在用这两个调用写不相关的内存。更改4为1- 您只阅读一项。

你从不使用num_bytes; 删除它。未使用的代码使思考已使用的代码变得更加困难。:)

您分配的内存是您需要的三倍：

                pixel =
                    (rgb *) malloc(sizeof(rgb) * bmpdata->alto *
                                   bmpdata->ancho * 3);

3看起来像是尝试在您的结构中解释红色、绿色、蓝色中的每一个，rgb但sizeof(rgb)已经知道结构的正确大小。（这可能是4字节，方便 32 位 CPU 对齐，也可能是12字节，再次对齐（每个都char在自己的4字节边界上），甚至可能是2464 位系统上真正喜欢处理对齐数据的字节8字节边界。）

我注意到的最后一件事：

                fread(pixel, 1, sizeof(rgb), fichero);

因为允许 C 编译器在结构中插入孔，所以您不能假设磁盘格式与您的内存结构定义相匹配。您需要使用GNU C 扩展属性，或者您需要使用为__packed__bmp格式设计的库或结构从中读取数据。如果这对您来说是一个有趣的项目，那么一定要尝试这__packed__条路线：如果有效，很好，如果无效，希望您能了解为什么不这样做，然后重新编写代码以手动加载结构的每个元素。如果您只是想获得可以正确解析位图的东西，那么您可能想尝试找到一些已经正确解析图像的预先编写的库。

（是的，正确解析图像非常重要； CVE 有一个格式错误的图像利用列表，允许攻击者控制程序，其中许多是可远程利用的。）

于 2011-03-19T00:53:42.070 回答