aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2025-02-02 21:25:45 +0100
committerMichael Niedermayer <michael@niedermayer.cc>2025-02-05 01:35:57 +0100
commit33679f5325f5ad82993eb2bda89ef5fe37f2e5da (patch)
treed9a95c7eb861ea3c002a5f31522800a09cde4321
parenta750bcb237bca9381c98fc40e83f272177c1fd7b (diff)
downloadffmpeg-33679f5325f5ad82993eb2bda89ef5fe37f2e5da.tar.gz
avcodec/vvc/refs: fix negative pps_scaling_win offsets
The spec seems to allow these to be negative Fixes: left shift of negative value -15 Fixes: 392687035/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VVC_fuzzer-6559804532785152 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Reviewed-by: Frank Plowman <post@frankplowman.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r--libavcodec/vvc/refs.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/libavcodec/vvc/refs.c b/libavcodec/vvc/refs.c
index 8d4b7bb35b..486515d06d 100644
--- a/libavcodec/vvc/refs.c
+++ b/libavcodec/vvc/refs.c
@@ -147,10 +147,10 @@ static VVCFrame *alloc_frame(VVCContext *s, VVCFrameContext *fc)
for (int j = 0; j < frame->ctb_count; j++)
frame->rpl_tab[j] = frame->rpl;
- win->left_offset = pps->r->pps_scaling_win_left_offset << sps->hshift[CHROMA];
- win->right_offset = pps->r->pps_scaling_win_right_offset << sps->hshift[CHROMA];
- win->top_offset = pps->r->pps_scaling_win_top_offset << sps->vshift[CHROMA];
- win->bottom_offset = pps->r->pps_scaling_win_bottom_offset << sps->vshift[CHROMA];
+ win->left_offset = pps->r->pps_scaling_win_left_offset * (1 << sps->hshift[CHROMA]);
+ win->right_offset = pps->r->pps_scaling_win_right_offset * (1 << sps->hshift[CHROMA]);
+ win->top_offset = pps->r->pps_scaling_win_top_offset * (1 << sps->vshift[CHROMA]);
+ win->bottom_offset = pps->r->pps_scaling_win_bottom_offset * (1 << sps->vshift[CHROMA]);
frame->ref_width = pps->r->pps_pic_width_in_luma_samples - win->left_offset - win->right_offset;
frame->ref_height = pps->r->pps_pic_height_in_luma_samples - win->bottom_offset - win->top_offset;