You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celix.apache.org by ab...@apache.org on 2013/09/19 11:18:09 UTC
svn commit: r1524664 -
/incubator/celix/trunk/shell_tui/private/src/shell_tui.c
Author: abroekhuis
Date: Thu Sep 19 09:18:08 2013
New Revision: 1524664
URL: http://svn.apache.org/r1524664
Log:
CELIX-85: Fixed an APR memory problem due to a thread not being stopped correctly.
Modified:
incubator/celix/trunk/shell_tui/private/src/shell_tui.c
Modified: incubator/celix/trunk/shell_tui/private/src/shell_tui.c
URL: http://svn.apache.org/viewvc/incubator/celix/trunk/shell_tui/private/src/shell_tui.c?rev=1524664&r1=1524663&r2=1524664&view=diff
==============================================================================
--- incubator/celix/trunk/shell_tui/private/src/shell_tui.c (original)
+++ incubator/celix/trunk/shell_tui/private/src/shell_tui.c Thu Sep 19 09:18:08 2013
@@ -144,11 +144,12 @@ celix_status_t bundleActivator_stop(void
status = bundleContext_removeServiceListener(context, act->listener);
if (status == CELIX_SUCCESS) {
+ apr_status_t join;
free(act->listener);
act->listener = NULL;
act->context = NULL;
act->running = false;
- apr_thread_detach(act->runnable);
+ apr_thread_join(&join, act->runnable);
}
return status;