This is an automated email from the ASF dual-hosted git repository.

gmurthy pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git


The following commit(s) were added to refs/heads/main by this push:
     new bdf1b9f  DISPATCH-2048: Fix shutdown leak of qd_deferred_call_t. This 
closes #1132.
bdf1b9f is described below

commit bdf1b9fa51fb53f0de66d331e49c0e6ff11a276c
Author: Ganesh Murthy <gmur...@apache.org>
AuthorDate: Fri Apr 16 14:38:52 2021 -0400

    DISPATCH-2048: Fix shutdown leak of qd_deferred_call_t. This closes #1132.
---
 src/server.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/server.c b/src/server.c
index 28f63f0..b407374 100644
--- a/src/server.c
+++ b/src/server.c
@@ -1386,7 +1386,9 @@ void qd_server_free(qd_server_t *qd_server)
             qd_session_cleanup(ctx);
             pn_connection_set_context(ctx->pn_conn, 0);
         }
-        if (ctx->free_user_id) free((char*)ctx->user_id);
+        invoke_deferred_calls(ctx, true);  // Discard any pending deferred 
calls
+        if (ctx->free_user_id)
+            free((char*)ctx->user_id);
         sys_mutex_free(ctx->deferred_call_lock);
         free(ctx->name);
         free(ctx->role);

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org
For additional commands, e-mail: commits-h...@qpid.apache.org

Reply via email to