From: José Bollo Date: Wed, 11 May 2016 15:42:49 +0000 (+0200) Subject: fixup session memory leak X-Git-Tag: blowfish_2.0.1~129 X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;h=f83af86907f072b8d58bc84acfb431682a9e3080;p=src%2Fapp-framework-binder.git fixup session memory leak Change-Id: I49ed17a3818be6107c46ce17533761858d99f735 Signed-off-by: José Bollo --- diff --git a/src/session.c b/src/session.c index 667738c4..22e50cfc 100644 --- a/src/session.c +++ b/src/session.c @@ -197,10 +197,11 @@ static void ctxStoreCleanUp (time_t now) // Loop on Sessions Table and remove anything that is older than timeout for (idx=0; idx < sessions.max; idx++) { - ctx = sessions.store[idx]; + ctx = ctxClientAddRef(sessions.store[idx]); if (ctx != NULL && ctxStoreTooOld(ctx, now)) { ctxClientClose (ctx); } + ctxClientUnref(ctx); } } @@ -276,6 +277,7 @@ void ctxClientUnref(struct AFB_clientCtx *clientCtx) --clientCtx->refcount; if (clientCtx->refcount == 0 && clientCtx->uuid[0] == 0) { ctxStoreDel (clientCtx); + free(clientCtx); } } } @@ -284,8 +286,12 @@ void ctxClientUnref(struct AFB_clientCtx *clientCtx) void ctxClientClose (struct AFB_clientCtx *clientCtx) { assert(clientCtx != NULL); - ctxUuidFreeCB (clientCtx); - clientCtx->uuid[0] = 0; + if (clientCtx->uuid[0] != 0) { + clientCtx->uuid[0] = 0; + ctxUuidFreeCB (clientCtx); + while(clientCtx->listeners != NULL) + ctxClientEventListenerRemove(clientCtx, clientCtx->listeners->listener); + } } // Sample Generic Ping Debug API