From dd4c6bc45c50b95ba819e7d75294fb9ca1a5bf3a Mon Sep 17 00:00:00 2001 From: Arun Raghavan Date: Tue, 3 Jul 2012 18:11:54 +0530 Subject: [PATCH] role-cork: Fix another minor leak Credit to David for finding this one. --- src/modules/module-role-cork.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/src/modules/module-role-cork.c b/src/modules/module-role-cork.c index 8306987e4..8f8b58dcd 100644 --- a/src/modules/module-role-cork.c +++ b/src/modules/module-role-cork.c @@ -225,9 +225,12 @@ int pa__init(pa_module *m) { if (roles) { const char *split_state = NULL; char *n = NULL; - while ((n = pa_split(roles, ",", &split_state))) + while ((n = pa_split(roles, ",", &split_state))) { if (n[0] != '\0') pa_idxset_put(u->trigger_roles, n, NULL); + else + pa_xfree(n); + } } if (pa_idxset_isempty(u->trigger_roles)) { pa_log_debug("Using role 'phone' as trigger role."); @@ -239,9 +242,12 @@ int pa__init(pa_module *m) { if (roles) { const char *split_state = NULL; char *n = NULL; - while ((n = pa_split(roles, ",", &split_state))) + while ((n = pa_split(roles, ",", &split_state))) { if (n[0] != '\0') pa_idxset_put(u->cork_roles, n, NULL); + else + pa_xfree(n); + } } if (pa_idxset_isempty(u->cork_roles)) { pa_log_debug("Using roles 'music' and 'video' as cork roles.");