You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by so...@apache.org on 2014/07/15 00:42:02 UTC

git commit: TS-1475: Fix Coverity # 1216790

Repository: trafficserver
Updated Branches:
  refs/heads/master 716a38273 -> 496615f3b


TS-1475: Fix Coverity # 1216790


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/496615f3
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/496615f3
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/496615f3

Branch: refs/heads/master
Commit: 496615f3b29fe7e89662835f52d3a1c44e1b5962
Parents: 716a382
Author: Phil Sorber <so...@apache.org>
Authored: Mon Jul 14 16:41:24 2014 -0600
Committer: Phil Sorber <so...@apache.org>
Committed: Mon Jul 14 16:41:51 2014 -0600

----------------------------------------------------------------------
 plugins/experimental/collapsed_connection/collapsed_connection.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/496615f3/plugins/experimental/collapsed_connection/collapsed_connection.cc
----------------------------------------------------------------------
diff --git a/plugins/experimental/collapsed_connection/collapsed_connection.cc b/plugins/experimental/collapsed_connection/collapsed_connection.cc
index 9e62b0f..9d082ce 100644
--- a/plugins/experimental/collapsed_connection/collapsed_connection.cc
+++ b/plugins/experimental/collapsed_connection/collapsed_connection.cc
@@ -943,12 +943,13 @@ collapsedConnectionMainHandler(TSCont /* contp ATS_UNUSED */, TSEvent event, voi
             addMutexRetry(txn_data, event, 0);
           } else {
             freeCcTxnData(txn_data);
+            txn_data = NULL;
           }
         } else {
           txn_data->cc_state = CC_REMOVE;
         }
       }
-      if (CC_INSERT == txn_data->cc_state || CC_REMOVE == txn_data->cc_state) {
+      if (txn_data && (CC_INSERT == txn_data->cc_state || CC_REMOVE == txn_data->cc_state)) {
         txn_data->cc_state = CC_REMOVE;
         if (TS_SUCCESS == updateOrRemoveHashEntry(txn_data)) {
           freeCcTxnData(txn_data);