Import fixes for some defects found by Coverity Scan from Red Hat's patchset.

diff --git a/src/HttpHdrRange.cc b/src/HttpHdrRange.cc
index f556d805a5136e03f3022b27a3fc28727e960e91..479e4f8e2529d0658dbfcb30e14c51c972061c6d 100644 (file)
--- a/src/HttpHdrRange.cc
+++ b/src/HttpHdrRange.cc
@@ -96,7 +96,7 @@ HttpHdrRangeSpec::parseInit(const char *field, int flen)
             return false;
     } else
         /* must have a '-' somewhere in _this_ field */
-        if (!((p = strchr(field, '-')) || (p - field >= flen))) {
+        if (!((p = strchr(field, '-')) && (p - field < flen))) {
             debugs(64, 2, "invalid (missing '-') range-spec near: '" << field << "'");
             return false;
         } else {

diff --git a/src/MemObject.cc b/src/MemObject.cc
index ecee3fdc7639ee0d7d3787d6b71ec178686b1b9e..59f8511822073af2e05648c4a8eb2fb2be874945 100644 (file)
--- a/src/MemObject.cc
+++ b/src/MemObject.cc
@@ -182,7 +182,7 @@ MemObject::dump() const
     debugs(20, DBG_IMPORTANT, "MemObject->nclients: " << nclients);
     debugs(20, DBG_IMPORTANT, "MemObject->reply: " << _reply);
     debugs(20, DBG_IMPORTANT, "MemObject->request: " << request);
-    debugs(20, DBG_IMPORTANT, "MemObject->log_url: " << log_url << " " << checkNullString(log_url));
+    debugs(20, DBG_IMPORTANT, "MemObject->log_url: " << checkNullString(log_url));
 }
 
 HttpReply const *

diff --git a/src/client_side.cc b/src/client_side.cc
index 3b9531f5213648485afb531b36790016a64d76be..54fcf1ab9d378d1bd77d1653deca62921f57c57b 100644 (file)
--- a/src/client_side.cc
+++ b/src/client_side.cc
@@ -714,7 +714,8 @@ ClientHttpRequest::logRequest()
         if (request)
             al->adapted_request = HTTPMSGLOCK(request);
         accessLogLog(al, checklist);
-        updateCounters();
+        if (request)
+            updateCounters();
 
         if (getConn() != NULL && getConn()->clientConnection != NULL)
             clientdbUpdate(getConn()->clientConnection->remote, logType, AnyP::PROTO_HTTP, out.size);