qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DISPATCH-1234) Router crash when edge tracking address handler tries to access freed endpoint
Date Thu, 03 Jan 2019 18:26:00 GMT

    [ https://issues.apache.org/jira/browse/DISPATCH-1234?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16733331#comment-16733331
] 

ASF GitHub Bot commented on DISPATCH-1234:
------------------------------------------

kgiusti commented on pull request #433: DISPATCH-1234 - Zero out the edge context on the link
before the edge…
URL: https://github.com/apache/qpid-dispatch/pull/433#discussion_r245089019
 
 

 ##########
 File path: src/router_core/modules/edge_addr_tracking/edge_addr_tracking.c
 ##########
 @@ -134,6 +135,10 @@ static void qdrc_address_endpoint_on_first_detach(void *link_context,
     qdrc_endpoint_detach_CT(endpoint_state->mc->core, endpoint_state->endpoint,
0);
     qdr_addr_tracking_module_context_t *mc = endpoint_state->mc;
     DEQ_REMOVE(mc->endpoint_state_list, endpoint_state);
+    endpoint_state->conn = 0;
+    endpoint_state->endpoint = 0;
+    endpoint_state->link->edge_context = 0;
 
 Review comment:
   Can (endpoint_state->link) be null?
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Router crash when edge tracking address handler tries to access freed endpoint
> ------------------------------------------------------------------------------
>
>                 Key: DISPATCH-1234
>                 URL: https://issues.apache.org/jira/browse/DISPATCH-1234
>             Project: Qpid Dispatch
>          Issue Type: Bug
>          Components: Container
>            Reporter: Ganesh Murthy
>            Assignee: Ganesh Murthy
>            Priority: Major
>             Fix For: 1.5.0
>
>
> {noformat}
> (gdb) bt
> #0  0x0000000004e3d53f in raise () from /lib64/libc.so.6
> #1  0x0000000004e27895 in abort () from /lib64/libc.so.6
> #2  0x0000000004e27769 in __assert_fail_base.cold.0 () from /lib64/libc.so.6
> #3  0x0000000004e359f6 in __assert_fail () from /lib64/libc.so.6
> #4  0x00000000048bc8d1 in on_addr_event (context=0x153a5e20, event=524288, addr=0x141158f0)
at /home/gmurthy/opensource/qpid-dispatch/src/router_core/modules/edge_addr_tracking/edge_addr_tracking.c:242
> #5  0x000000000489cbf0 in qdrc_event_addr_raise (core=0x15395d30, event=524288, addr=0x141158f0)
at /home/gmurthy/opensource/qpid-dispatch/src/router_core/core_events.c:114
> #6  0x00000000048ac5e3 in qdr_core_unbind_address_link_CT (core=0x15395d30, addr=0x141158f0,
link=0x1410fda0) at /home/gmurthy/opensource/qpid-dispatch/src/router_core/router_core.c:504
> #7  0x000000000489c110 in qdr_link_inbound_detach_CT (core=0x15395d30, action=0x13c411c0,
discard=false) at /home/gmurthy/opensource/qpid-dispatch/src/router_core/connections.c:1642
> #8  0x00000000048add55 in router_core_thread (arg=0x15395d30) at /home/gmurthy/opensource/qpid-dispatch/src/router_core/router_core_thread.c:124
> #9  0x000000000496458e in start_thread () from /lib64/libpthread.so.0
> #10 0x0000000004f026a3 in clone () from /lib64/libc.so.6
> (gdb){noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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


Mime
View raw message