Skip to content
Snippets Groups Projects
Commit a3870581 authored by George Kiagiadakis's avatar George Kiagiadakis
Browse files

spa-device / m-monitor: fix pw_proxy leaks reported by pw_core

1. device export proxies must be destroyed manually since they are
   not associated with the WpRegistry
2. the monitors should not disconnect before all WpSpaDevice objects
   are destroyed; remove the manual disconnect call and let GObject
   ref counting do its job (the core will disconnect when its last ref
   count is dropped after the last monitor plugin is destroyed)
parent b59ff481
No related branches found
No related tags found
No related merge requests found
......@@ -270,6 +270,7 @@ wp_spa_device_finalize (GObject * object)
{
WpSpaDevice *self = WP_SPA_DEVICE (object);
g_clear_pointer (&self->proxy, pw_proxy_destroy);
self->device = NULL;
g_clear_pointer (&self->handle, pw_unload_spa_handle);
g_clear_pointer (&self->properties, wp_properties_unref);
......
......@@ -393,7 +393,6 @@ wp_monitor_deactivate (WpPlugin * plugin)
{
WpMonitor *self = WP_MONITOR (plugin);
wp_core_disconnect (self->local_core);
g_clear_object (&self->monitor);
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment