From d4fc9187371590cf04c8c6475c4a7cb4b65fba2d Mon Sep 17 00:00:00 2001 From: Wim Taymans Date: Mon, 9 Nov 2020 16:49:49 +0100 Subject: [PATCH] module-adapter: don't remove the listener in destroy Or else we won't receive the free event anymore and we leak the adapter. --- src/modules/module-adapter.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/modules/module-adapter.c b/src/modules/module-adapter.c index ed92d10ea..e59e9a502 100644 --- a/src/modules/module-adapter.c +++ b/src/modules/module-adapter.c @@ -91,7 +91,6 @@ static void node_destroy(void *data) struct node_data *nd = data; pw_log_debug(NAME" %p: destroy %p", nd, nd->adapter); spa_list_remove(&nd->link); - spa_hook_remove(&nd->adapter_listener); nd->adapter = NULL; } @@ -99,6 +98,7 @@ static void node_free(void *data) { struct node_data *nd = data; pw_log_debug(NAME" %p: free %p", nd, nd->follower); + spa_hook_remove(&nd->adapter_listener); pw_impl_node_destroy(nd->follower); }