C++ memcpy 和 happy 访问冲突

Question

由于某种原因，我无法想象我遇到了访问冲突。

memcpy_s (buffer, bytes_per_line * height, image, bytes_per_line * height);

这是整个函数：

int Flip_Bitmap(UCHAR *image, int bytes_per_line, int height)   
{   
    // this function is used to flip bottom-up .BMP images   

    UCHAR *buffer; // used to perform the image processing   
    int index;     // looping index   

    // allocate the temporary buffer   
    if (!(buffer = (UCHAR *) malloc (bytes_per_line * height)))   
        return(0);   

    // copy image to work area   
    //memcpy(buffer, image, bytes_per_line * height);   
    memcpy_s (buffer, bytes_per_line * height, image, bytes_per_line * height);

    // flip vertically   
    for (index = 0; index < height; index++)   
        memcpy(&image[((height - 1) - index) * bytes_per_line], &buffer[index * bytes_per_line], bytes_per_line);   

    // release the memory   
    free(buffer);   

    // return success   
    return(1);   

} // end Flip_Bitmap   

整个代码： http://pastebin.com/udRqgCfU

要运行此程序，您需要在源目录中使用 24 位位图。 这是较大代码的一部分，我正在尝试使 Load_Bitmap_File 函数正常工作...... 那么，有什么想法吗？

Answer 1

您遇到访问冲突是因为许多图像程序未正确设置biSizeImage。您正在使用的图像可能将 biSizeImage 设置为 0，因此您不会为图像数据分配任何内存（实际上，您可能会分配 4-16 个字节，因为大多数malloc 实现也会返回非 NULL 值。因此，当您复制数据时，您正在读取该数组的末尾，这会导致访问冲突。

忽略 biSizeImage 参数并自行计算图像大小。请记住，每个扫描线的大小必须是 4 字节的倍数，因此您需要向上舍入：

// Pseudocode
#define ROUNDUP(value, power_of_2) (((value) + (power_of_2) - 1) & (~((power_of_2) - 1)))
bytes_per_line = ROUNDUP(width * bits_per_pixel/8, 4)
image_size = bytes_per_line * height;

然后只需使用相同的图像大小来读取图像数据并翻转它。

Answer 2

正如评论所说，图像数据不一定是 width*height*bytes_per_pixel

内存访问通常在 32 位边界上更快，并且在处理图像时速度通常很重要。因此，图像的行通常会移位到从 4 字节（32 位）边界开始

如果图像像素是 32 位（即 RGBA），这不是问题，但如果每个像素有 3 字节（24 位颜色），那么对于某些情况图像宽度，其中列数 * 3 不是 4 的倍数，则将在每行的 edn 处插入额外的空白字节。

图像格式可能有一个“步幅”宽度或 elemsize 值来告诉您这一点。

Answer 3

您为 image 分配了 bitmap->bitmapinfoheader.biSizeImage，但继续复制 bitmap->bitmapinfoheader.biWidth * (bitmap->bitmapinfoheader.biBitCount / 8 ) * bitmap->bitmapinfoheader.biHeight 字节数据。我敢打赌这两个数字不一样。