Hi!
Attached patch is one of two patches necessary to fix ticket #1304.
The MSDN documentation indicates that the palette is not part of the
bitmapinfoheader but the bitmapinfoheader is part of the bitmapinfo
structure that also contains the palette.
An alternative is to add the extradata_size except if it is a palette
but imo, attached is more correct.
The second issue to fix ticket #1304 is that a too small frame size
is written to the bitmapinfoheader, it is 192x127 for a 189x127 pal8
frame. 0 is allowed if the pix_fmt is rawvideo.
Please comment, Carl Eugen
diff --git a/libavformat/riffenc.c b/libavformat/riffenc.c
index 2eb2ae1..db35af4 100644
--- a/libavformat/riffenc.c
+++ b/libavformat/riffenc.c
@@ -214,7 +214,7 @@ void ff_put_bmp_header(AVIOContext *pb, AVCodecContext *enc,
int extradata_size = enc->extradata_size - 9*keep_height;
/* size */
- avio_wl32(pb, 40 + (ignore_extradata ? 0 :extradata_size));
+ avio_wl32(pb, 40);
avio_wl32(pb, enc->width);
//We always store RGB TopDown
avio_wl32(pb, enc->codec_tag || keep_height ? enc->height : -enc->height);
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
