From fb95ce9bfc80167eba843567aa62d62d359a2bf1 Mon Sep 17 00:00:00 2001 From: Vineeth TM Date: Thu, 20 Aug 2015 17:20:50 +0900 Subject: [PATCH] gst-omx: Fix memory leaks when context parse fails When g_option_context_parse fails, context and error variables are not getting free'd which results in memory leaks. Free'ing the same. And replacing g_error_free with g_clear_error, which checks if the error being passed is not NULL and sets the variable to NULL on free'ing. https://bugzilla.gnome.org/show_bug.cgi?id=753865 --- examples/egl/testegl.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/examples/egl/testegl.c b/examples/egl/testegl.c index bccf6c190e..99aa81059b 100644 --- a/examples/egl/testegl.c +++ b/examples/egl/testegl.c @@ -1495,6 +1495,8 @@ main (int argc, char **argv) g_option_context_add_group (ctx, gst_init_get_option_group ()); if (!g_option_context_parse (ctx, &argc, &argv, &err)) { g_print ("Error initializing: %s\n", GST_STR_NULL (err->message)); + g_option_context_free (ctx); + g_clear_error (&err); exit (1); } g_option_context_free (ctx);