From 30ea0e47b5ddb13b431f997655b567d738e86d23 Mon Sep 17 00:00:00 2001
From: William Yu <genwillyu@gmail.com>
Date: Thu, 11 Aug 2011 15:47:33 +0200
Subject: [PATCH] vf_select: add NULL-checks in uninit

Fix NULL pointer dereference in case of failed initialization.

Signed-off-by: Stefano Sabatini <stefano.sabatini-lala@poste.it>
---
 libavfilter/vf_select.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/libavfilter/vf_select.c b/libavfilter/vf_select.c
index 3cd2347fa4..33269b6594 100644
--- a/libavfilter/vf_select.c
+++ b/libavfilter/vf_select.c
@@ -319,14 +319,18 @@ static av_cold void uninit(AVFilterContext *ctx)
     AVFilterBufferRef *picref;
     int i;
 
+    if (select->expr)
     av_expr_free(select->expr);
     select->expr = NULL;
 
+    if (select->pending_frames) {
     for (i = 0; i < av_fifo_size(select->pending_frames)/sizeof(picref); i++) {
         av_fifo_generic_read(select->pending_frames, &picref, sizeof(picref), NULL);
         avfilter_unref_buffer(picref);
     }
     av_fifo_free(select->pending_frames);
+    }
+    select->pending_frames = NULL;
 }
 
 AVFilter avfilter_vf_select = {