Question

我正在尝试设计一个将位图图像加载到内存中并最终显示它的函数。我正在使用Watcom 16位C编译器编译代码，目标设置为DOS。我在DOSBox中运行程序。代码如下：

#ifndef __BITMAP_H__
#define __BITMAP_H__

#include <stdio.h>
#include <stdlib.h>

typedef struct DIB
{
    int header_size;
    int px_width;
    int px_height;
}DIB_t;

DIB_t *load_bitmap(char *file_name)
{
    FILE *bmp_file;
    DIB_t *bitmap;
    char garbage[4];
    int c, file_size, pixel_offset;

    bitmap = (DIB_t*)malloc(sizeof bitmap);

    if(!bitmap)
    {
        perror("malloc()");
        return NULL;
    }

    bmp_file = fopen(file_name, "rb");

    if(!bmp_file)
    {
        perror("fopen()");
        return NULL;
    }

    c = fgetc(bmp_file);

    if(c == 'B')
    {
        c = fgetc(bmp_file);

        if(c == 'M')
        {
            fread(&file_size, 4, 1, bmp_file);
            fread(garbage, 1, 4, bmp_file);
            fread(&pixel_offset, 4, 1, bmp_file);
            fread(&bitmap->header_size, 4, 1, bmp_file);
            fread(&bitmap->px_width, 4, 1, bmp_file);
            fread(&bitmap->px_height, 4, 1, bmp_file);
            printf("BMP width: %dpx\nBMP Height: %dpx", bitmap->px_width, bitmap->px_height);
            fclose(bmp_file);
            return bitmap;
        }
    }

    fputs("File format not supported.\n", stderr);
    fclose(bmp_file);
    return NULL;
}
#endif

当你运行这个程序时，它将输出：“BMP width：％dpx \ n”，但在换行符之后没有输出？我觉得这很奇怪。我已经确认没有任何操作失败或设置errno，并且px_height实际上设置为它的适当值。这是你们有经验的吗？

Answer 1

你只是用这条线来指示足够的空间用于指针

bitmap = (DIB_t*)malloc(sizeof(bitmap));

你真正需要的是

bitmap = (DIB_t*)malloc(sizeof(DIB_t));

另外，正如mrbatch所说，你使用的是16位编译器，因此尝试将4字节数据读入2字节int变量。确保sizeof（）与您正在阅读的内容相匹配，例如：一个long int

所以 - 一般来说 - 你通过编写比你应该写的更多的数据来破坏你的堆栈和你的堆，并且你可以期望你的程序表现得非常奇怪：）

Answer 2

除了保罗为您的sizeof更正malloc的答案外，请尝试在声明中将int值更改为long（我假设您是关于包含4个字节值的位图的代码是正确的）。 16位编译器中的int将为2个字节。对于4字节值，您需要long（或等效long int）。

<小时/> 正如评论中所建议的那样，更好的方法是包含inttypes.h并使用int32_t代替int或long来确保32位值。

调用fread后奇怪的printf行为

2 个答案: