You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by su...@apache.org on 2015/01/15 01:32:20 UTC

trafficserver git commit: [TS-3294]: Fix unchecked return value coverity defect

Repository: trafficserver
Updated Branches:
  refs/heads/master eaf53d5e8 -> 0e88a2f7d


[TS-3294]: Fix unchecked return value coverity defect

Coverity CID#1237970


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

Branch: refs/heads/master
Commit: 0e88a2f7d0f895a1b0854617f9729a05a4262915
Parents: eaf53d5
Author: Sudheer Vinukonda <su...@yahoo-inc.com>
Authored: Thu Jan 15 00:31:22 2015 +0000
Committer: Sudheer Vinukonda <su...@yahoo-inc.com>
Committed: Thu Jan 15 00:31:22 2015 +0000

----------------------------------------------------------------------
 lib/atscppapi/src/Transaction.cc    | 2 +-
 lib/atscppapi/src/utils_internal.cc | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/0e88a2f7/lib/atscppapi/src/Transaction.cc
----------------------------------------------------------------------
diff --git a/lib/atscppapi/src/Transaction.cc b/lib/atscppapi/src/Transaction.cc
index 52f3dfe..b1ff159 100644
--- a/lib/atscppapi/src/Transaction.cc
+++ b/lib/atscppapi/src/Transaction.cc
@@ -69,7 +69,7 @@ Transaction::Transaction(void *raw_txn) {
   TSHttpTxn txn = static_cast<TSHttpTxn>(raw_txn);
   TSMBuffer hdr_buf;
   TSMLoc hdr_loc;
-  TSHttpTxnClientReqGet(txn, &hdr_buf, &hdr_loc);
+  (void) TSHttpTxnClientReqGet(txn, &hdr_buf, &hdr_loc);
   if (!hdr_buf || !hdr_loc) {
     LOG_ERROR("TSHttpTxnClientReqGet tshttptxn=%p returned a null hdr_buf=%p or hdr_loc=%p.", txn, hdr_buf, hdr_loc);
   }

http://git-wip-us.apache.org/repos/asf/trafficserver/blob/0e88a2f7/lib/atscppapi/src/utils_internal.cc
----------------------------------------------------------------------
diff --git a/lib/atscppapi/src/utils_internal.cc b/lib/atscppapi/src/utils_internal.cc
index 040ffb0..236def8 100644
--- a/lib/atscppapi/src/utils_internal.cc
+++ b/lib/atscppapi/src/utils_internal.cc
@@ -56,7 +56,7 @@ int handleTransactionEvents(TSCont cont, TSEvent event, void *edata) {
     // This is here to force a refresh of the cached client request url
     TSMBuffer hdr_buf;
     TSMLoc hdr_loc;
-    TSHttpTxnClientReqGet(static_cast<TSHttpTxn>(transaction.getAtsHandle()), &hdr_buf, &hdr_loc);
+    (void) TSHttpTxnClientReqGet(static_cast<TSHttpTxn>(transaction.getAtsHandle()), &hdr_buf, &hdr_loc);
     break;
   case TS_EVENT_HTTP_SEND_REQUEST_HDR:
     utils::internal::initTransactionServerRequest(transaction);