mirror of
https://gitlab.freedesktop.org/pulseaudio/pulseaudio.git
synced 2025-10-29 05:40:23 -04:00
alsa-mixer: Don't use dangling pointers as port hashmap keys
If alsa settings/options were used, the string to construct the name was freed, leading to dangling pointers to strings inside the hashmap. BugLink: https://bugs.launchpad.net/bugs/932804 Signed-off-by: David Henningsson <david.henningsson@canonical.com>
This commit is contained in:
parent
e91e78bb54
commit
13a84d0747
1 changed files with 2 additions and 2 deletions
|
|
@ -4397,7 +4397,7 @@ static pa_device_port* device_port_alsa_init(pa_hashmap *ports,
|
|||
|
||||
p = pa_device_port_new(core, name, description, sizeof(pa_alsa_port_data));
|
||||
pa_assert(p);
|
||||
pa_hashmap_put(ports, name, p);
|
||||
pa_hashmap_put(ports, p->name, p);
|
||||
p->profiles = pa_hashmap_new(pa_idxset_string_hash_func, pa_idxset_string_compare_func);
|
||||
|
||||
data = PA_DEVICE_PORT_DATA(p);
|
||||
|
|
@ -4412,7 +4412,7 @@ static pa_device_port* device_port_alsa_init(pa_hashmap *ports,
|
|||
pa_hashmap_put(p->profiles, cp->name, cp);
|
||||
|
||||
if (extra) {
|
||||
pa_hashmap_put(extra, name, p);
|
||||
pa_hashmap_put(extra, p->name, p);
|
||||
pa_device_port_ref(p);
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue