h264dec: use a large enough field for reference list modification values

Message ID 20170818130041.01D3D5DACB@aruru.libav.org
State New
Headers show

Commit Message

Janne Grunau Aug. 18, 2017, 1 p.m.
Module: libav
Branch: master
Commit: f70f71d60c7ae88c19078a48dc6e0789b78c7300

Author:    Anton Khirnov <anton@khirnov.net>
Committer: Anton Khirnov <anton@khirnov.net>
Date:      Thu Aug 17 12:15:58 2017 +0200

h264dec: use a large enough field for reference list modification values

pic_num can be at most 17-bit, so uint8_t is not sufficient.

Found-By: Bradley Sepos <bradley@bradleysepos.com>
CC: libav-stable@libav.org

---

 libavcodec/h264dec.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Patch

diff --git a/libavcodec/h264dec.h b/libavcodec/h264dec.h
index ddfe224..cce5e19 100644
--- a/libavcodec/h264dec.h
+++ b/libavcodec/h264dec.h
@@ -268,7 +268,7 @@  typedef struct H264SliceContext {
                                          *   according to picture reordering in slice header */
     struct {
         uint8_t op;
-        uint8_t val;
+        uint32_t val;
     } ref_modifications[2][32];
     int nb_ref_modifications[2];