validate: fix double free

Summary:
Move variable declarations in the for block so we won't try re-free
tldir in case of early short circuiting of the 'for' code.

Depends on D348

Reviewers: thiblahute

Reviewed By: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D349
This commit is contained in:
Guillaume Desmottes 2015-10-10 10:51:10 +01:00 committed by Thibault Saunier
parent 5067cee05b
commit 9005428910

View file

@ -2303,7 +2303,6 @@ gst_validate_scenario_load (GstValidateScenario * scenario,
{
gchar **scenarios = NULL;
guint i;
gchar *lfilename = NULL, *tldir = NULL;
gboolean found_actions = FALSE, is_config, ret = TRUE;
const gchar *scenarios_path = g_getenv ("GST_VALIDATE_SCENARIOS_PATH");
@ -2316,6 +2315,7 @@ gst_validate_scenario_load (GstValidateScenario * scenario,
scenarios = g_strsplit (scenario_name, ":", -1);
for (i = 0; scenarios[i]; i++) {
gchar *lfilename = NULL, *tldir = NULL;
/* First check if the scenario name is not a full path to the
* actual scenario */