[trafficserver] branch master updated: cachekey: handle empty regex group captures

[gancho]

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

gancho 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 4d5790f  cachekey: handle empty regex group captures
4d5790f is described below

commit 4d5790f796c90db9b8ee245ff179a5a5f66c2608
Author: Gancho Tenev <gan...@apache.org>
AuthorDate: Sat Aug 11 14:31:12 2018 -0700

    cachekey: handle empty regex group captures
---
 plugins/cachekey/pattern.cc | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/plugins/cachekey/pattern.cc b/plugins/cachekey/pattern.cc
index a63fc97..319a4a3 100644
--- a/plugins/cachekey/pattern.cc
+++ b/plugins/cachekey/pattern.cc
@@ -303,6 +303,12 @@ Pattern::replace(const String &subject, String &result)
     int start     = ovector[2 * replIndex];
     int length    = ovector[2 * replIndex + 1] - ovector[2 * replIndex];
 
+    /* Handle the case when no match / a group capture result in an empty 
string */
+    if (start < 0) {
+      start  = 0;
+      length = 0;
+    }
+
     String src(_replacement, _tokenOffset[i], 2);
     String dst(subject, start, length);