Am 30.06.17 um 18:43 schrieb Paul B Mahol:
> Fixes #5297.
>
> Signed-off-by: Paul B Mahol <one...@gmail.com>
> ---
> libavcodec/alsdec.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/alsdec.c b/libavcodec/alsdec.c
> index d95e30d..a925502 100644
> --- a/libavcodec/alsdec.c
> +++ b/libavcodec/alsdec.c
> @@ -705,8 +705,8 @@ static int read_var_block_data(ALSDecContext *ctx,
> ALSBlockData *bd)
> } else {
> *bd->opt_order = sconf->max_order;
> }
> - if (*bd->opt_order > bd->block_length) {
> - *bd->opt_order = bd->block_length;
> + if (*bd->opt_order > sconf->max_order) {
> + *bd->opt_order = sconf->max_order;
> av_log(avctx, AV_LOG_ERROR, "Predictor order too large.\n");
> return AVERROR_INVALIDDATA;
> }
This check will never fire because in all cases this check has already been
applied or opt_order is explicitly set to equal max_order. See code above.
Paul, seriously, are you just trying to get me even more busy?
-Thilo
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
