mirrors/pulseaudio
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pulseaudio/pulseaudio.git synced 2025-10-29 05:40:23 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

Revert "Revert "role-cork: Fix a minor leak""

Browse source 
This reverts commit a2c4bf22e0.

As Tanu points out, I'm stupid. The pa_xstrdup(n) was unnecessary
because pa_split() allocates a new string for us.
This commit is contained in:
Arun Raghavan 2012-07-03 18:04:48 +05:30
parent b7f7eb5657
commit ded196f93a
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/modules/module-role-cork.c
View file

@ -227,7 +227,7 @@ int pa__init(pa_module *m) {
char *n = NULL;
while ((n = pa_split(roles, ",", &split_state)))
if (n[0] != '\0')
pa_idxset_put(u->trigger_roles, pa_xstrdup(n), NULL);
pa_idxset_put(u->trigger_roles, n, NULL);
}
if (pa_idxset_isempty(u->trigger_roles)) {
pa_log_debug("Using role 'phone' as trigger role.");
@ -241,7 +241,7 @@ int pa__init(pa_module *m) {
char *n = NULL;
while ((n = pa_split(roles, ",", &split_state)))
if (n[0] != '\0')
pa_idxset_put(u->cork_roles, pa_xstrdup(n), NULL);
pa_idxset_put(u->cork_roles, n, NULL);
}
if (pa_idxset_isempty(u->cork_roles)) {
pa_log_debug("Using roles 'music' and 'video' as cork roles.");