[trafficserver] branch master updated: Fix a nullptr dereference

[ma...@apache.org]

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

maskit 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 10fd8db  Fix a nullptr dereference
10fd8db is described below

commit 10fd8db9a0eea4d72af0761d2b4f3881f4f80e11
Author: Masakazu Kitajo <ma...@apache.org>
AuthorDate: Mon May 7 15:32:11 2018 +0100

    Fix a nullptr dereference
---
 proxy/hdrs/HttpCompat.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/proxy/hdrs/HttpCompat.cc b/proxy/hdrs/HttpCompat.cc
index 8340bde..4ce1008 100644
--- a/proxy/hdrs/HttpCompat.cc
+++ b/proxy/hdrs/HttpCompat.cc
@@ -454,7 +454,8 @@ HttpCompat::do_header_values_rfc2068_14_43_match(MIMEField *hdr1, MIMEField *hdr
   const char *hdr2_val = iter2.get_first(hdr2, &hdr2_val_len);
 
   while (hdr1_val || hdr2_val) {
-    if (hdr1_val_len != hdr2_val_len || ParseRules::strncasecmp_eow(hdr1_val, hdr2_val, hdr1_val_len) == false) {
+    if (!hdr1_val || !hdr2_val || hdr1_val_len != hdr2_val_len ||
+        ParseRules::strncasecmp_eow(hdr1_val, hdr2_val, hdr1_val_len) == false) {
       return false;
     }
 

-- 
To stop receiving notification emails like this one, please contact
maskit@apache.org.