ucm: fix use-after-free in parse_component_seq()

Signed-off-by: Jaroslav Kysela <perex@perex.cz>
alsa-project · Jun 8, 2020 · 6eb78d4 · 6eb78d4
1 parent 141acab
commit 6eb78d4
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/ucm/parser.c b/src/ucm/parser.c
@@ -617,11 +617,12 @@ static int parse_component_seq(snd_use_case_mgr_t *uc_mgr,
 		return err;
 
 	cmpt_seq->device = find_component_dev(uc_mgr, val);
-	free(val);
 	if (!cmpt_seq->device) {
 		uc_error("error: Cannot find component device %s", val);
+		free(val);
 		return -EINVAL;
 	}
+	free(val);
 
 	/* Parent needs its enable or disable sequence */
 	cmpt_seq->enable = enable;