You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2012/11/28 16:26:50 UTC

[3/3] git commit: TS-1506 % log symbol will crash TS when requesting a SSL url. Review: leif

TS-1506 %<cquuh> log symbol will crash TS when requesting a SSL url.
Review: leif


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/4c1c9675
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/4c1c9675
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/4c1c9675

Branch: refs/heads/master
Commit: 4c1c9675530d27901eee39e5c98928c324d6bf8f
Parents: 17e6c89
Author: Conan Wang <co...@gmail.com>
Authored: Wed Nov 28 08:20:59 2012 -0700
Committer: Leif Hedstrom <zw...@apache.org>
Committed: Wed Nov 28 08:20:59 2012 -0700

----------------------------------------------------------------------
 proxy/logging/LogAccessHttp.cc |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/4c1c9675/proxy/logging/LogAccessHttp.cc
----------------------------------------------------------------------
diff --git a/proxy/logging/LogAccessHttp.cc b/proxy/logging/LogAccessHttp.cc
index 026f07e..d198df9 100644
--- a/proxy/logging/LogAccessHttp.cc
+++ b/proxy/logging/LogAccessHttp.cc
@@ -210,7 +210,7 @@ LogAccessHttp::validate_unmapped_url_path(void)
 
     if (m_client_req_unmapped_url_path_len >= 6) {      // xxx:// - minimum schema size
       c = (char *) memchr((void *) m_client_req_unmapped_url_path_str, ':', m_client_req_unmapped_url_path_len - 1);
-      if (c && (len = (int) (c - m_client_req_unmapped_url_path_str)) <= 4) {   // 4 - max schema size
+      if (c && (len = (int) (c - m_client_req_unmapped_url_path_str)) <= 5) {   // 5 - max schema size
         if (len + 2 <= m_client_req_unmapped_url_canon_len && c[1] == '/' && c[2] == '/') {
           len += 3;             // Skip "://"
           m_client_req_unmapped_url_host_str = &m_client_req_unmapped_url_canon_str[len];