[FFmpeg-cvslog] avformat/riffenc: correct calculation for extradata size

Gyan Doshi git at videolan.org
Sun Jul 26 08:40:43 EEST 2020


ffmpeg | branch: master | Gyan Doshi <ffmpeg at gyani.pro> | Sat Jul 25 10:57:32 2020 +0530| [93694fe777d12a9432b5ceeb564631da6311ae26] | committer: Gyan Doshi

avformat/riffenc: correct calculation for extradata size

In 1ec2b3de5a, the extradata size was affected when the raster was
signaled as flipped due to user-set option rather than via extradata.
This resulted in a wrong header size being written. Fixed.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=93694fe777d12a9432b5ceeb564631da6311ae26
---

 libavformat/riffenc.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/libavformat/riffenc.c b/libavformat/riffenc.c
index d0ee98bfcc..04a21fcffa 100644
--- a/libavformat/riffenc.c
+++ b/libavformat/riffenc.c
@@ -209,10 +209,10 @@ int ff_put_wav_header(AVFormatContext *s, AVIOContext *pb,
 void ff_put_bmp_header(AVIOContext *pb, AVCodecParameters *par,
                        int for_asf, int ignore_extradata, int rgb_frame_is_flipped)
 {
-    int keep_height = (par->extradata_size >= 9 &&
-                      !memcmp(par->extradata + par->extradata_size - 9, "BottomUp", 9)) ||
-                      rgb_frame_is_flipped;
-    int extradata_size = par->extradata_size - 9*keep_height;
+    int flipped_extradata = (par->extradata_size >= 9 &&
+                            !memcmp(par->extradata + par->extradata_size - 9, "BottomUp", 9));
+    int keep_height = flipped_extradata || rgb_frame_is_flipped;
+    int extradata_size = par->extradata_size - 9*flipped_extradata;
     enum AVPixelFormat pix_fmt = par->format;
     int pal_avi;
 



More information about the ffmpeg-cvslog mailing list