Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PR] avcodec/adpcm: fix integer overflow in N64 (PR #21753)
@ 2026-02-14  1:47 michaelni via ffmpeg-devel
  0 siblings, 0 replies; only message in thread
From: michaelni via ffmpeg-devel @ 2026-02-14  1:47 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: michaelni

PR #21753 opened by michaelni
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21753
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21753.patch

Fixes: signed integer overflow: 1077919680 + 1077936128 cannot be represented in type 'int'
Fixes: 471686763/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ADPCM_N64_DEC_fuzzer-6493712281829376

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>


>From a802ef96e5e6e96976c5443f81fd92f06f8bb540 Mon Sep 17 00:00:00 2001
From: Michael Niedermayer <michael@niedermayer.cc>
Date: Sat, 14 Feb 2026 01:32:40 +0100
Subject: [PATCH] avcodec/adpcm: fix integer overflow in N64

Fixes: signed integer overflow: 1077919680 + 1077936128 cannot be represented in type 'int'
Fixes: 471686763/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ADPCM_N64_DEC_fuzzer-6493712281829376

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 libavcodec/adpcm.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c
index 8ddca46b67..1502a5cb04 100644
--- a/libavcodec/adpcm.c
+++ b/libavcodec/adpcm.c
@@ -2688,7 +2688,8 @@ static int adpcm_decode_frame(AVCodecContext *avctx, AVFrame *frame,
                 int16_t *sf_out = &out[j*8];
 
                 for (int i = 0; i < 8; i++) {
-                    int sample, delta = 0;
+                    int sample;
+                    unsigned delta = 0;
 
                     for (int o = 0; o < order; o++)
                         delta += coefs[o*8 + i] * hist[(8 - order) + o];
@@ -2699,7 +2700,7 @@ static int adpcm_decode_frame(AVCodecContext *avctx, AVFrame *frame,
                     }
 
                     sample = sf_codes[i] * 2048;
-                    sample = (sample + delta) / 2048;
+                    sample = (int)(sample + delta) / 2048;
                     sample = av_clip_int16(sample);
                     sf_out[i] = sample;
                 }
-- 
2.52.0

_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2026-02-14  1:47 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-02-14  1:47 [FFmpeg-devel] [PR] avcodec/adpcm: fix integer overflow in N64 (PR #21753) michaelni via ffmpeg-devel

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git