[FFmpeg-cvslog] avcodec/vaapi: mask unused bits in bitplane_present.value

Jerome Borsboom git at videolan.org
Mon Mar 5 00:38:00 EET 2018

ffmpeg | branch: master | Jerome Borsboom <jerome.borsboom at carpalis.nl> | Sun Feb 25 20:13:46 2018 +0100| [0ec7eb930520b21b9cda34809378b0a8ca3a46e2] | committer: Mark Thompson

avcodec/vaapi: mask unused bits in bitplane_present.value

Due to the union construct, unused bits in bitplane_present.value might
be uninitialized even when the used bits are all set to a value.
Masking the unused bits prevents spurious true values when all used
bits are unset, e.g. skipped pictures.

Signed-off-by: Jerome Borsboom <jerome.borsboom at carpalis.nl>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=0ec7eb930520b21b9cda34809378b0a8ca3a46e2

 libavcodec/vaapi_vc1.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/vaapi_vc1.c b/libavcodec/vaapi_vc1.c
index 97b23917f6..74ba783141 100644
--- a/libavcodec/vaapi_vc1.c
+++ b/libavcodec/vaapi_vc1.c
@@ -383,7 +383,7 @@ static int vaapi_vc1_start_frame(AVCodecContext *avctx, av_unused const uint8_t
     if (err)
         goto fail;
-    if (pic_param.bitplane_present.value) {
+    if (pic_param.bitplane_present.value & 0x7f) {
         uint8_t *bitplane;
         const uint8_t *ff_bp[3];
         int x, y, n;

More information about the ffmpeg-cvslog mailing list