summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2023-11-30 02:36:41 +0100
committerMichael Niedermayer <michael@niedermayer.cc>2023-12-05 12:38:16 +0100
commit22daf2148fc072f8f347af939f88b3af7896ab60 (patch)
tree0790fec07258a646ec10bceb4ebdcd7317cb33c9
parent4cdf2c7f768015c74078544d153f243b6d9b9ac5 (diff)
avcodec/av1dec: Fix resolving zero divisor
Fixes: Out of array read Fixes: global-buffer-overflow-AV1 Found-by: "Leonelli, Matteo" <matteo.leonelli@cispa.de> Tested-by: "Wang, Fei W" <fei.w.wang@intel.com> Reviewed-by: "Wang, Fei W" <fei.w.wang@intel.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r--libavcodec/av1dec.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/libavcodec/av1dec.c b/libavcodec/av1dec.c
index 6114cb78e6..4dcde234c6 100644
--- a/libavcodec/av1dec.c
+++ b/libavcodec/av1dec.c
@@ -177,7 +177,7 @@ static uint8_t get_shear_params_valid(AV1DecContext *s, int idx)
int16_t alpha, beta, gamma, delta, divf, divs;
int64_t v, w;
int32_t *param = &s->cur_frame.gm_params[idx][0];
- if (param[2] < 0)
+ if (param[2] <= 0)
return 0;
alpha = av_clip_int16(param[2] - (1 << AV1_WARPEDMODEL_PREC_BITS));