qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [qpid-dispatch] kgiusti commented on a change in pull request #545: DISPATCH-1391 - Zero out pn_link references on all qd_links related t…
Date Wed, 24 Jul 2019 17:19:27 GMT
kgiusti commented on a change in pull request #545: DISPATCH-1391 - Zero out pn_link references
on all qd_links related t…
URL: https://github.com/apache/qpid-dispatch/pull/545#discussion_r306927274
 
 

 ##########
 File path: src/container.c
 ##########
 @@ -541,14 +541,16 @@ void qd_container_handle_event(qd_container_t *container, pn_event_t
*event,
                     qd_conn->n_sessions--;
                 }
 
-                pn_link = pn_link_head(conn, PN_LOCAL_ACTIVE | PN_REMOTE_CLOSED);
+                //Sweep thru every pn_link in this connection and a matching session and
zero out the
+                // qd_link->pn_link reference. We do this in order to not miss any pn_links
+                pn_link = pn_link_head(conn, 0);
 
 Review comment:
   Suggestion:
   
   Since you now have to check every link on the session why not combine this loop with the
previous into a single loop?
   Loop through all once using an if statement to check for PN_LOCAL_ACTIVE | PN_REMOTE_ACTIVE
and do the detach handling  zeroing the pn_link entry.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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


With regards,
Apache Git Services

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


Mime
View raw message