aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOlivier CrĂȘte <olivier.crete@collabora.com>2021-08-24 09:43:42 -0400
committerPaul B Mahol <onemda@gmail.com>2021-08-26 09:20:56 +0200
commit521388edb7d3c176b444bbc3a42723cbafab2d55 (patch)
tree98e1fdb1991b3cd816f83228b510a8c402d73681
parentc1c7f2b61f30a2a7c3a3a9bfa892dfc3b2bda21c (diff)
downloadffmpeg-521388edb7d3c176b444bbc3a42723cbafab2d55.tar.gz
avcodec/wma: Return specific error code
This way, the calling function can just forward it instead of making it up. Signed-off-by: Olivier CrĂȘte <olivier.crete@collabora.com>
-rw-r--r--libavcodec/wma.c4
-rw-r--r--libavcodec/wmaprodec.c13
2 files changed, 10 insertions, 7 deletions
diff --git a/libavcodec/wma.c b/libavcodec/wma.c
index 004ffb5515..d9fda2dbfa 100644
--- a/libavcodec/wma.c
+++ b/libavcodec/wma.c
@@ -459,7 +459,7 @@ int ff_wma_run_level_decode(AVCodecContext *avctx, GetBitContext *gb,
if (get_bits1(gb)) {
av_log(avctx, AV_LOG_ERROR,
"broken escape sequence\n");
- return -1;
+ return AVERROR_INVALIDDATA;
} else
offset += get_bits(gb, frame_len_bits) + 4;
} else
@@ -477,7 +477,7 @@ int ff_wma_run_level_decode(AVCodecContext *avctx, GetBitContext *gb,
offset,
num_coefs
);
- return -1;
+ return AVERROR_INVALIDDATA;
}
return 0;
diff --git a/libavcodec/wmaprodec.c b/libavcodec/wmaprodec.c
index e0d00d2d37..3e7ec1a8b9 100644
--- a/libavcodec/wmaprodec.c
+++ b/libavcodec/wmaprodec.c
@@ -991,13 +991,16 @@ static int decode_coeffs(WMAProDecodeCtx *s, int c)
/** decode run level coded coefficients */
if (cur_coeff < s->subframe_len) {
+ int ret;
+
memset(&ci->coeffs[cur_coeff], 0,
sizeof(*ci->coeffs) * (s->subframe_len - cur_coeff));
- if (ff_wma_run_level_decode(s->avctx, &s->gb, vlc,
- level, run, 1, ci->coeffs,
- cur_coeff, s->subframe_len,
- s->subframe_len, s->esc_len, 0))
- return AVERROR_INVALIDDATA;
+ ret = ff_wma_run_level_decode(s->avctx, &s->gb, vlc,
+ level, run, 1, ci->coeffs,
+ cur_coeff, s->subframe_len,
+ s->subframe_len, s->esc_len, 0);
+ if (ret < 0)
+ return ret;
}
return 0;