summaryrefslogtreecommitdiff
path: root/avplay.c
diff options
context:
space:
mode:
authorAlex Converse <alex.converse@gmail.com>2012-04-10 18:18:53 -0700
committerAlex Converse <alex.converse@gmail.com>2012-04-11 10:24:01 -0700
commite1ce756844e684876318570dcebc74bc66c084f0 (patch)
tree45ab1347760846cf1116342a5083e08a29028871 /avplay.c
parentebbede22657a17c40f913857b23c8de85e75fe5c (diff)
avplay: Don't free video filters string until the end of decoding.
av_freep()ing inside configure_video_filters() leaves a dangling reference in the calling code, and the filter string is needed again when reconfiguring video filters for a size change.
Diffstat (limited to 'avplay.c')
-rw-r--r--avplay.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/avplay.c b/avplay.c
index d1cf53d117..488d0fcd13 100644
--- a/avplay.c
+++ b/avplay.c
@@ -1731,7 +1731,6 @@ static int configure_video_filters(AVFilterGraph *graph, VideoState *is, const c
if ((ret = avfilter_graph_parse(graph, vfilters, inputs, outputs, NULL)) < 0)
return ret;
- av_freep(&vfilters);
} else {
if ((ret = avfilter_link(filt_src, 0, filt_out, 0)) < 0)
return ret;
@@ -1831,6 +1830,7 @@ static int video_thread(void *arg)
}
the_end:
#if CONFIG_AVFILTER
+ av_freep(&vfilters);
avfilter_graph_free(&graph);
#endif
av_free(frame);