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

[Marton Balint]

On Sat, 17 Dec 2016, pkoshevoy at gmail.com wrote:

> From: Pavel Koshevoy <pkoshevoy at gmail.com>
>
> Steps to reproduce:
> ./ffmpeg_g -f s16be -i /dev/null -af atempo=0.5 -y /tmp/atempo.wav
> ---
> libavfilter/af_atempo.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavfilter/af_atempo.c b/libavfilter/af_atempo.c
> index 59b08ec..a487882 100644
> --- a/libavfilter/af_atempo.c
> +++ b/libavfilter/af_atempo.c
> @@ -914,8 +914,8 @@ static int yae_flush(ATempoContext *atempo,
>
>     atempo->state = YAE_FLUSH_OUTPUT;
> 
> -    if (atempo->position[0] == frag->position[0] + frag->nsamples &&
> -        atempo->position[1] == frag->position[1] + frag->nsamples) {
> +    if (atempo->position[0] >= frag->position[0] + frag->nsamples &&
> +        atempo->position[1] >= frag->position[1] + frag->nsamples) {
>         // the current fragment is already flushed:
>         return 0;
>     }

Thanks, this indeed fixes the assertion I came accross. Do you want me to 
apply?

Regards,
Marton