[trafficserver] branch master updated: coverity 1199995 : return code not checked

[bc...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

bcall pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git

The following commit(s) were added to refs/heads/master by this push:
       new  bb59ae6   coverity 1199995 : return code not checked
bb59ae6 is described below

commit bb59ae6a0daf34a8b6d94f5d62186ae1a5810b58
Author: Jason Kenny <dr...@live.com>
AuthorDate: Thu May 11 13:54:26 2017 -0400

    coverity 1199995 : return code not checked
---
 lib/cppapi/ClientRequest.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/cppapi/ClientRequest.cc b/lib/cppapi/ClientRequest.cc
index f951975..ebbe651 100644
--- a/lib/cppapi/ClientRequest.cc
+++ b/lib/cppapi/ClientRequest.cc
@@ -60,9 +60,9 @@ const Url &
 atscppapi::ClientRequest::getPristineUrl() const
 {
   if (!state_->pristine_url_loc_) {
-    TSHttpTxnPristineUrlGet(state_->txn_, &state_->pristine_hdr_buf_, &state_->pristine_url_loc_);
+    TSReturnCode ret = TSHttpTxnPristineUrlGet(state_->txn_, &state_->pristine_hdr_buf_, &state_->pristine_url_loc_);
 
-    if ((state_->pristine_hdr_buf_ != nullptr) && (state_->pristine_url_loc_ != nullptr)) {
+    if ((state_->pristine_hdr_buf_ != nullptr) && (state_->pristine_url_loc_ != nullptr) && ret == TS_SUCCESS) {
       state_->pristine_url_.init(state_->pristine_hdr_buf_, state_->pristine_url_loc_);
       LOG_DEBUG("Pristine URL initialized");
     } else {

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].