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 2013/03/31 03:14:37 UTC

[18/51] git commit: TS-1660 Host field should not has c style terminator

TS-1660 Host field should not has c style terminator


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

Branch: refs/heads/3.3.x
Commit: 8d6562ba44b408b48192b4b6951c669b3236dda0
Parents: 0f43630
Author: weijin <weilogster at 126 dot com>
Authored: Wed Mar 20 14:34:13 2013 -0600
Committer: Leif Hedstrom <zw...@apache.org>
Committed: Wed Mar 20 14:34:13 2013 -0600

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


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/8d6562ba/proxy/http/HttpTransact.cc
----------------------------------------------------------------------
diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc
index e5f2394..931fe80 100644
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -5254,7 +5254,7 @@ HttpTransact::RequestError_t HttpTransact::check_request_validity(State* s, HTTP
     return MISSING_HOST_FIELD;
   }
 
-  if (hostname_len >= MAXDNAME || hostname_len <= 0) {
+  if (hostname_len >= MAXDNAME || hostname_len <= 0 || memchr(hostname, '\0', hostname_len)) {
     return BAD_HTTP_HEADER_SYNTAX;
   }