qpid-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From gmur...@apache.org
Subject [qpid-dispatch] branch master updated: DISPATCH-1214 - Free the error object on second detaches
Date Fri, 14 Dec 2018 22:29:44 GMT
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
     new ccb8585  DISPATCH-1214 - Free the error object on second detaches
ccb8585 is described below

commit ccb8585011cdbce112ecdd3b3af4e300d30e5889
Author: Ganesh Murthy <gmurthy@redhat.com>
AuthorDate: Fri Dec 14 17:29:21 2018 -0500

    DISPATCH-1214 - Free the error object on second detaches
---
 src/router_core/core_link_endpoint.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/router_core/core_link_endpoint.c b/src/router_core/core_link_endpoint.c
index e1867f7..532188c 100644
--- a/src/router_core/core_link_endpoint.c
+++ b/src/router_core/core_link_endpoint.c
@@ -210,11 +210,15 @@ void qdrc_endpoint_do_detach_CT(qdr_core_t *core, qdrc_endpoint_t *ep,
qdr_error
     if (ep->link->detach_count == 1) {
         if (!!ep->desc->on_first_detach)
             ep->desc->on_first_detach(ep->link_context, error);
-        else
+        else {
             qdr_link_outbound_detach_CT(core, ep->link, 0, QDR_CONDITION_NONE, true);
+            qdr_error_free(error);
+        }
     } else {
         if (!!ep->desc->on_second_detach)
             ep->desc->on_second_detach(ep->link_context, error);
+        else
+            qdr_error_free(error);
         ep->link->core_endpoint = 0;
         free_qdrc_endpoint_t(ep);
     }


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


Mime
View raw message