diff options
author | Ruijing Dong <ruijing.dong@amd.com> | 2022-11-02 15:35:53 -0400 |
---|---|---|
committer | Haihao Xiang <haihao.xiang@intel.com> | 2022-11-04 09:51:37 +0800 |
commit | 89e3814569dc061cd4008351ed3d87366645960e (patch) | |
tree | c4c07d6de04708cf03568256db05ff709ff76e64 /libavcodec/dca_lbr.c | |
parent | 3819719099df601c470e961b9d49b9100c65641b (diff) | |
download | ffmpeg-89e3814569dc061cd4008351ed3d87366645960e.tar.gz |
avcodec/av1_vaapi: fixed a decoding corruption issue
In av1_spec.pdf page 38/669, there is a sentence below:
if ( frame_type == KEY_FRAME && show_frame ) {
for ( i = 0; i < NUM_REF_FRAMES; i++) {
RefValid[ i ] = 0
......
}
......
}
This shows that the condition of invalidating current
DPB frames should be the coming frame_type is KEY_FRAME plus
show_frame is equal to 1. Otherwise, some of the frames
in sequence after KEY_FRAME still refer to the reference frames
before KEY_FRAME, and if these before KEY_FRAME reference
frames were invalidated, these frames could not find their
reference frames, and it could cause image corruption.
Mesa fix is in https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/19386
Reviewed-by: Fei Wang <fei.w.wang@intel.com>
Signed-off-by: Ruijing Dong <ruijing.dong@amd.com>
Diffstat (limited to 'libavcodec/dca_lbr.c')
0 files changed, 0 insertions, 0 deletions