diff mbox

[FFmpeg-devel,03/12] af_tempo: Add missing error check

Message ID 20170706182842.7785-4-derek.buitenhuis@gmail.com
State Accepted
Commit 704b774ae0290dd8f2f30ba7c6f7427d0c6180c6
Headers show

Commit Message

Derek Buitenhuis July 6, 2017, 6:28 p.m. UTC 
Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
---
 libavfilter/af_atempo.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Rostislav Pehlivanov July 6, 2017, 9:16 p.m. UTC | #1 
On 6 July 2017 at 19:28, Derek Buitenhuis <derek.buitenhuis@gmail.com>
wrote:

> Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
> ---
>  libavfilter/af_atempo.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/libavfilter/af_atempo.c b/libavfilter/af_atempo.c
> index 76410221d6..944df1dd32 100644
> --- a/libavfilter/af_atempo.c
> +++ b/libavfilter/af_atempo.c
> @@ -1148,6 +1148,8 @@ static int request_frame(AVFilterLink *outlink)
>
>              if (n_out) {
>                  ret = push_samples(atempo, outlink, n_out);
> +                if (ret < 0)
> +                    return ret;
>              }
>          }
>
> --
> 2.11.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

LGTM
diff mbox

Patch

diff --git a/libavfilter/af_atempo.c b/libavfilter/af_atempo.c
index 76410221d6..944df1dd32 100644
--- a/libavfilter/af_atempo.c
+++ b/libavfilter/af_atempo.c
@@ -1148,6 +1148,8 @@  static int request_frame(AVFilterLink *outlink)
 
             if (n_out) {
                 ret = push_samples(atempo, outlink, n_out);
+                if (ret < 0)
+                    return ret;
             }
         }