[trafficserver] 02/02: Delete space after semi-colon when deleting cookies with a test to ensure not overflowing the cookies char buffer. Ensure idx + 1 is less than len before checking for space.

[zwoop]

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

zwoop pushed a commit to branch 7.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git

commit 2efadb84e072e22bab600caae524b24af432fd21
Author: Steven Feltner <sfelt...@godaddy.com>
AuthorDate: Mon Dec 18 07:17:19 2017 -0700

    Delete space after semi-colon when deleting cookies with a test to ensure 
not overflowing the cookies char buffer.  Ensure idx + 1 is less than len 
before checking for space.
    
    (cherry picked from commit 08b560d57ac6c30f7c3ab11e796efbd11c9ec669)
---
 plugins/header_rewrite/operators.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plugins/header_rewrite/operators.cc 
b/plugins/header_rewrite/operators.cc
index fc818ad..d467f89 100644
--- a/plugins/header_rewrite/operators.cc
+++ b/plugins/header_rewrite/operators.cc
@@ -796,7 +796,7 @@ CookieHelper::cookieModifyHelper(const char *cookies, const 
size_t cookies_len,
         }
         // If we have not reached the end and there is a space after the
         // semi-colon, advance one char
-        if (idx < cookies_len && std::isspace(cookies[idx + 1])) {
+        if (idx + 1 < cookies_len && std::isspace(cookies[idx + 1])) {
           idx++;
         }
         // cookie value is found

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