[FFmpeg-devel] [PATCH 2/2] lavfi/atempo: Avoid false triggering an assertion failure

Marton Balint cus at passwd.hu
Tue Sep 5 22:01:34 EEST 2017


On Sun, 3 Sep 2017, pkoshevoy at gmail.com wrote:

> From: Pavel Koshevoy <pkoshevoy at gmail.com>
>
> Steps to reproduce:
> 1. revert 4240e5b047379b29c33dd3f4438bc4e610527b83
> 2. ./ffmpeg -f lavfi -i sine=d=1 -af aselect=e=0,atempo=0.5 -y atempo.wav
> ---
> libavfilter/af_atempo.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/libavfilter/af_atempo.c b/libavfilter/af_atempo.c
> index 9eee8a63a8..41c8c0382a 100644
> --- a/libavfilter/af_atempo.c
> +++ b/libavfilter/af_atempo.c
> @@ -914,6 +914,11 @@ static int yae_flush(ATempoContext *atempo,
>
>     atempo->state = YAE_FLUSH_OUTPUT;
> 
> +    if (!atempo->nfrag) {
> +      // there is nothing to flush:
> +      return 0;
> +    }
> +
>     if (atempo->position[0] == frag->position[0] + frag->nsamples &&
>         atempo->position[1] == frag->position[1] + frag->nsamples) {
>         // the current fragment is already flushed:

Thanks, this does the trick. Will apply in 1-2 days.

Regards,
Marton


More information about the ffmpeg-devel mailing list