fdo-mirrors/wireplumber
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pipewire/wireplumber.git synced 2025-12-20 04:10:03 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

m-mixer-api: Fix memory in leak wp_mixer_api_set_volume

Browse source 
Declare result from wp_object_manager_lookup as g_autoptr, to prevent
leaking memory.
This commit is contained in:
Torkel Niklasson 2024-09-26 12:01:18 +02:00
parent b68a6794cd
commit 255b65d182
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
modules/module-mixer-api.c
View file

@ -501,7 +501,7 @@ wp_mixer_api_set_volume (WpMixerApi * self, guint32 id, GVariant * vvolume)
props = wp_spa_pod_builder_end (b); props = wp_spa_pod_builder_end (b);
if (info->device_id != SPA_ID_INVALID) { if (info->device_id != SPA_ID_INVALID) {
WpPipewireObject *device = wp_object_manager_lookup (self->om, g_autoptr (WpPipewireObject) device = wp_object_manager_lookup (self->om,
WP_TYPE_DEVICE, WP_CONSTRAINT_TYPE_G_PROPERTY, WP_TYPE_DEVICE, WP_CONSTRAINT_TYPE_G_PROPERTY,
"bound-id", "=u", info->device_id, NULL); "bound-id", "=u", info->device_id, NULL);
g_return_val_if_fail (device != NULL, FALSE); g_return_val_if_fail (device != NULL, FALSE);
@ -514,7 +514,7 @@ wp_mixer_api_set_volume (WpMixerApi * self, guint32 id, GVariant * vvolume)
"save", "b", true, "save", "b", true,
NULL)); NULL));
} else { } else {
WpPipewireObject *node = wp_object_manager_lookup (self->om, g_autoptr (WpPipewireObject) node = wp_object_manager_lookup (self->om,
WP_TYPE_NODE, WP_CONSTRAINT_TYPE_G_PROPERTY, WP_TYPE_NODE, WP_CONSTRAINT_TYPE_G_PROPERTY,
"bound-id", "=u", id, NULL); "bound-id", "=u", id, NULL);
g_return_val_if_fail (node != NULL, FALSE); g_return_val_if_fail (node != NULL, FALSE);