tools: gst-launch: fix minor memory leak when failing to parse options
authorAntonio Ospite <ao2@ao2.it>
Wed, 2 Nov 2016 12:57:51 +0000 (13:57 +0100)
committerSebastian Dröge <sebastian@centricular.com>
Fri, 4 Nov 2016 14:59:36 +0000 (16:59 +0200)
Commit 215cfcf99338 (gstreamer: Fix memory leaks when context parse
fails) fixes some memory leak, but in one of the newly added calls to
g_clear_error() the wrong variable was passed.

When failing to parse command line options, free the "err" variable, not
the "error" one.

https://bugzilla.gnome.org/show_bug.cgi?id=773907

tools/gst-launch.c

index 2de754d..006eeae 100644 (file)
@@ -1015,7 +1015,7 @@ main (int argc, char *argv[])
       g_printerr ("Error initializing: %s\n", GST_STR_NULL (err->message));
     else
       g_printerr ("Error initializing: Unknown error!\n");
-    g_clear_error (&error);
+    g_clear_error (&err);
     g_option_context_free (ctx);
     exit (1);
   }