diff options
author | David Rosca <nowrep@gmail.com> | 2023-11-19 11:27:29 +0100 |
---|---|---|
committer | Eric Engestrom <eric@engestrom.ch> | 2023-11-24 17:15:18 +0000 |
commit | 142c333748e052b47bdfd2c224ae905bbd24d41c (patch) | |
tree | de340db811d7c5011cffd343aa7fe88c223a360a /src/util | |
parent | c05aadcf25acfdf259dc27001aa9cc4332f385e4 (diff) | |
download | mesa-142c333748e052b47bdfd2c224ae905bbd24d41c.tar.gz mesa-142c333748e052b47bdfd2c224ae905bbd24d41c.tar.bz2 mesa-142c333748e052b47bdfd2c224ae905bbd24d41c.zip |
util/rbsp: Fill bits twice if reading more than 16 bits
vl_rbsp_fillbits may fill less than 32 bits if it removes emulation
prevention bytes, but will fill at least 16 bits. We need to call it
twice when reading more than 16 bits.
This fixes parsing H264 SPS packed header in va frontend when emulation
prevention bytes are at position where 32 bit values are read.
Cc: mesa-stable
Reviewed-by: Leo Liu <leo.liu@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26276>
(cherry picked from commit 73d69ef1e6b013f923ec459c789976570baaf447)
Diffstat (limited to 'src/util')
-rw-r--r-- | src/util/vl_rbsp.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/util/vl_rbsp.h b/src/util/vl_rbsp.h index 0842b0f1490..82043fc8752 100644 --- a/src/util/vl_rbsp.h +++ b/src/util/vl_rbsp.h @@ -138,6 +138,8 @@ static inline unsigned vl_rbsp_u(struct vl_rbsp *rbsp, unsigned n) return 0; vl_rbsp_fillbits(rbsp); + if (n > 16) + vl_rbsp_fillbits(rbsp); return vl_vlc_get_uimsbf(&rbsp->nal, n); } |