You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by tr...@apache.org on 2017/10/04 14:48:48 UTC

[4/9] qpid-dispatch git commit: DISPATCH-847 - Fixed Coverity issue 181413 (Null-check after dereference)

DISPATCH-847 - Fixed Coverity issue 181413 (Null-check after dereference)


Project: http://git-wip-us.apache.org/repos/asf/qpid-dispatch/repo
Commit: http://git-wip-us.apache.org/repos/asf/qpid-dispatch/commit/938cdfaf
Tree: http://git-wip-us.apache.org/repos/asf/qpid-dispatch/tree/938cdfaf
Diff: http://git-wip-us.apache.org/repos/asf/qpid-dispatch/diff/938cdfaf

Branch: refs/heads/master
Commit: 938cdfafa291388d9fc29e8050bb3723ca742d35
Parents: f293015
Author: Ted Ross <tr...@redhat.com>
Authored: Wed Oct 4 10:03:48 2017 -0400
Committer: Ted Ross <tr...@redhat.com>
Committed: Wed Oct 4 10:03:48 2017 -0400

----------------------------------------------------------------------
 src/router_core/transfer.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/qpid-dispatch/blob/938cdfaf/src/router_core/transfer.c
----------------------------------------------------------------------
diff --git a/src/router_core/transfer.c b/src/router_core/transfer.c
index dcd7563..baaec53 100644
--- a/src/router_core/transfer.c
+++ b/src/router_core/transfer.c
@@ -1072,7 +1072,6 @@ static void qdr_deliver_continue_CT(qdr_core_t *core, qdr_action_t *action, bool
 
     qdr_deliver_continue_peers_CT(core, in_dlv);
 
-
     if (qd_message_receive_complete(qdr_delivery_message(in_dlv))) {
         //
         // The entire message has now been received. Check to see if there are in process subscriptions that need to
@@ -1081,7 +1080,7 @@ static void qdr_deliver_continue_CT(qdr_core_t *core, qdr_action_t *action, bool
         qdr_subscription_t *sub = DEQ_HEAD(in_dlv->subscriptions);
         while (sub) {
             DEQ_REMOVE_HEAD(in_dlv->subscriptions);
-            qdr_forward_on_message_CT(core, sub, in_dlv ? in_dlv->link : 0, in_dlv->msg);
+            qdr_forward_on_message_CT(core, sub, in_dlv->link, in_dlv->msg);
             sub = DEQ_HEAD(in_dlv->subscriptions);
         }
 


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