diff mbox series

[FFmpeg-devel] avcodec/wmadec: Don't check for errors for complete VLC tables

Message ID 20201022184543.667852-1-andreas.rheinhardt@gmail.com
State Accepted
Commit 2b702015d8b1e273280f9389e7accf68266d0165
Headers show
Series [FFmpeg-devel] avcodec/wmadec: Don't check for errors for complete VLC tables | expand

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/PPC64_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Andreas Rheinhardt Oct. 22, 2020, 6:45 p.m. UTC 
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
---
 libavcodec/wmadec.c | 9 ---------
 1 file changed, 9 deletions(-)

Comments

Michael Niedermayer Oct. 23, 2020, 8:06 a.m. UTC | #1 
On Thu, Oct 22, 2020 at 08:45:43PM +0200, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
> ---
>  libavcodec/wmadec.c | 9 ---------
>  1 file changed, 9 deletions(-)

probably ok

[...]
diff mbox series

Patch

diff --git a/libavcodec/wmadec.c b/libavcodec/wmadec.c
index 07fd960f7f..8504d8d6c4 100644
--- a/libavcodec/wmadec.c
+++ b/libavcodec/wmadec.c
@@ -345,10 +345,6 @@  static int decode_exp_vlc(WMACodecContext *s, int ch)
 
     while (q < q_end) {
         code = get_vlc2(&s->gb, s->exp_vlc.table, EXPVLCBITS, EXPMAX);
-        if (code < 0) {
-            av_log(s->avctx, AV_LOG_ERROR, "Exponent vlc invalid\n");
-            return -1;
-        }
         /* NOTE: this offset is the same as MPEG-4 AAC! */
         last_exp += code - 60;
         if ((unsigned) last_exp + 60 >= FF_ARRAY_ELEMS(pow_tab)) {
@@ -560,11 +556,6 @@  static int wma_decode_block(WMACodecContext *s)
                         } else {
                             code = get_vlc2(&s->gb, s->hgain_vlc.table,
                                             HGAINVLCBITS, HGAINMAX);
-                            if (code < 0) {
-                                av_log(s->avctx, AV_LOG_ERROR,
-                                       "hgain vlc invalid\n");
-                                return -1;
-                            }
                             val += code - 18;
                         }
                         s->high_band_values[ch][i] = val;