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 8ce3593 DISPATCH-1697: Delete the connection ref associated with
alt_conn_id. This closes #1124.
8ce3593 is described below
commit 8ce359345b56b449b84b9d0d0c7365341700f89f
Author: Ganesh Murthy <gmur...@apache.org>
AuthorDate: Wed Apr 14 14:47:42 2021 -0400
DISPATCH-1697: Delete the connection ref associated with alt_conn_id. This
closes #1124.
---
src/alloc_pool.c | 3 ---
src/router_core/router_core.c | 5 +++++
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/alloc_pool.c b/src/alloc_pool.c
index 768c3f2..6cdd5f8 100644
--- a/src/alloc_pool.c
+++ b/src/alloc_pool.c
@@ -91,10 +91,7 @@ DEQ_DECLARE(qd_alloc_type_t, qd_alloc_type_list_t);
static const char *leaking_types[] = {
// DISPATCH-1679:
"qd_timer_t", "qd_connector_t",
-
"qd_hash_handle_t", // DISPATCH-1696
- "qdr_conn_identifier_t", // DISPATCH-1697
- "qdr_connection_ref_t", // DISPATCH-1698
// system_tests_edge_router (centos7)
// DISPATCH-1699
diff --git a/src/router_core/router_core.c b/src/router_core/router_core.c
index 3f391e0..83e687b 100644
--- a/src/router_core/router_core.c
+++ b/src/router_core/router_core.c
@@ -263,6 +263,11 @@ void qdr_core_free(qdr_core_t *core)
qdr_route_check_id_for_deletion_CT(core, conn->conn_id);
}
+ if (conn->alt_conn_id) {
+ qdr_del_connection_ref(&conn->alt_conn_id->connection_refs, conn);
+ qdr_route_check_id_for_deletion_CT(core, conn->alt_conn_id);
+ }
+
qdr_connection_work_t *work = DEQ_HEAD(conn->work_list);
while (work) {
DEQ_REMOVE_HEAD(conn->work_list);
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org
For additional commands, e-mail: commits-h...@qpid.apache.org
