diff mbox

[FFmpeg-devel,11/12] af_amix: Add missing error check

Message ID 20170706182842.7785-12-derek.buitenhuis@gmail.com
State Accepted
Commit b198e0913854ddeefb4335a2a12c422cfc5aaf12
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_amix.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Paul B Mahol July 6, 2017, 9:24 p.m. UTC | #1 
On 7/6/17, Derek Buitenhuis <derek.buitenhuis@gmail.com> wrote:
> Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
> ---
>  libavfilter/af_amix.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/libavfilter/af_amix.c b/libavfilter/af_amix.c
> index f19e6b39a5..809c8bf9ca 100644
> --- a/libavfilter/af_amix.c
> +++ b/libavfilter/af_amix.c
> @@ -468,6 +468,8 @@ static int filter_frame(AVFilterLink *inlink, AVFrame
> *buf)
>
>      ret = av_audio_fifo_write(s->fifos[i], (void **)buf->extended_data,
>                                buf->nb_samples);
> +    if (ret < 0)
> +        goto fail;
>
>      av_frame_free(&buf);
>      return output_frame(outlink, 0);
> --
> 2.11.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

ok
diff mbox

Patch

diff --git a/libavfilter/af_amix.c b/libavfilter/af_amix.c
index f19e6b39a5..809c8bf9ca 100644
--- a/libavfilter/af_amix.c
+++ b/libavfilter/af_amix.c
@@ -468,6 +468,8 @@  static int filter_frame(AVFilterLink *inlink, AVFrame *buf)
 
     ret = av_audio_fifo_write(s->fifos[i], (void **)buf->extended_data,
                               buf->nb_samples);
+    if (ret < 0)
+        goto fail;
 
     av_frame_free(&buf);
     return output_frame(outlink, 0);