diff mbox series

[FFmpeg-devel] avcodec/adpcm: only process right samples when decoding stereo

Message ID d0f37ecfd62e9a6f2cbaab67fa51e4e48ea28094.1721803857.git.pross@xvid.org
State New
Headers show
Series [FFmpeg-devel] avcodec/adpcm: only process right samples when decoding stereo | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Peter Ross July 24, 2024, 6:51 a.m. UTC 
Fixes Coverity issue #1610760.
---
 libavcodec/adpcm.c | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)
diff mbox series

Patch

diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c
index afdbeaa15e..58f2ef0dfb 100644
--- a/libavcodec/adpcm.c
+++ b/libavcodec/adpcm.c
@@ -1684,22 +1684,24 @@  static int adpcm_decode_frame(AVCodecContext *avctx, AVFrame *frame,
             for (int count2 = 0; count2 < (channels == 2 ? 28 : 14); count2++) {
                 byte = bytestream2_get_byteu(&gb);
                 next_left_sample  = sign_extend(byte >> 4, 4) * (1 << shift_left);
-                next_right_sample = sign_extend(byte,      4) * (1 << shift_right);
 
                 next_left_sample = (next_left_sample +
                     (current_left_sample * coeff1l) +
                     (previous_left_sample * coeff2l) + 0x80) >> 8;
-                next_right_sample = (next_right_sample +
-                    (current_right_sample * coeff1r) +
-                    (previous_right_sample * coeff2r) + 0x80) >> 8;
 
                 previous_left_sample = current_left_sample;
                 current_left_sample = av_clip_int16(next_left_sample);
-                previous_right_sample = current_right_sample;
-                current_right_sample = av_clip_int16(next_right_sample);
                 *samples++ = current_left_sample;
 
                 if (channels == 2){
+                    next_right_sample = sign_extend(byte, 4) * (1 << shift_right);
+
+                    next_right_sample = (next_right_sample +
+                        (current_right_sample * coeff1r) +
+                        (previous_right_sample * coeff2r) + 0x80) >> 8;
+
+                    previous_right_sample = current_right_sample;
+                    current_right_sample = av_clip_int16(next_right_sample);
                     *samples++ = current_right_sample;
                 } else {
                     next_left_sample  = sign_extend(byte, 4) * (1 << shift_left);