Commit 4876fe04 authored by Ohn Yongjin's avatar Ohn Yongjin Committed by Tim-Philipp Müller
Browse files

tests: audioconvert: Fix memory leak in failure path

Don't set a bad example by leaking things, even if calling
g_error() will make the process abort.

https://bugzilla.gnome.org/show_bug.cgi?id=783418
parent d36a2255
......@@ -467,8 +467,9 @@ verify_convert (const gchar * which, void *in, int inlength,
/* positions for 1 and 2 channels are implicit if not provided */
if (out_chans > 2 && gst_structure_has_field (in_s, "channel-positions")) {
if (!gst_structure_has_field (out_s, "channel-positions")) {
g_error ("Channel layout got lost somewhere:\n\nIns : %s\nOuts: %s\n",
gst_structure_to_string (in_s), gst_structure_to_string (out_s));
gst_printerrln ("In : %" GST_PTR_FORMAT, in_s);
gst_printerrln ("Out : %" GST_PTR_FORMAT, out_s);
g_error ("Channel layout got lost somewhere\n");
}
}
gst_caps_unref (ccaps);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment