From 882b74cf79088a8368842cc0cd1b672114ff5ffa Mon Sep 17 00:00:00 2001 From: David Robillard Date: Fri, 11 May 2012 22:33:28 +0000 Subject: Fix suil_instance_free to avoid a double free when the wrapper has already destructed. git-svn-id: http://svn.drobilla.net/lad/trunk/suil@4359 a436a847-0d15-0410-975c-d299462d15a1 --- src/instance.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'src/instance.c') diff --git a/src/instance.c b/src/instance.c index 57cb7e1..fcb7786 100644 --- a/src/instance.c +++ b/src/instance.c @@ -257,13 +257,18 @@ suil_instance_free(SuilInstance* instance) } free(instance->features); + // Call wrapper free function to destroy widgets and drop references + if (instance->wrapper && instance->wrapper->free) { + instance->wrapper->free(instance->wrapper); + } + + // Call cleanup to destroy UI (if it still exists at this point) if (instance->handle) { instance->descriptor->cleanup(instance->handle); } + + // Close libraries and free everything if (instance->wrapper) { - if (instance->wrapper->free) { - instance->wrapper->free(instance->wrapper); - } dlclose(instance->wrapper->lib); free(instance->wrapper); } -- cgit v1.2.1