diff options
author | corvid <corvid@lavabit.com> | 2010-02-25 21:15:45 +0000 |
---|---|---|
committer | corvid <corvid@lavabit.com> | 2010-02-25 21:15:45 +0000 |
commit | ae6386f8c9def1c247b99902d1315ceb5c20b31b (patch) | |
tree | 7e557acb4d339d3852f72fca597cdfafecec3f50 /src | |
parent | 0bd335403e7793551bb76cbdf629fbba05fdfecc (diff) |
don't accept ' ' between http header field name and ':'
It's a bit unclear from rfc2616. The httpbis folks found
(http://wiki.tools.ietf.org/wg/httpbis/trac/ticket/30)
that those implementations that did accept it generally
treated the space as part of the name, so it has been
disallowed.
The current relevant bit of the httpbis draft spec says exactly
"No whitespace is allowed between the header field name and colon."
Diffstat (limited to 'src')
-rw-r--r-- | src/cache.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/cache.c b/src/cache.c index 0116fbf5..7c1d2991 100644 --- a/src/cache.c +++ b/src/cache.c @@ -585,7 +585,6 @@ static char *Cache_parse_field(const char *header, const char *fieldname) } i += j; - while (header[i] == ' ') i++; if (header[i] == ':') { /* Field found! */ while (header[++i] == ' ' || header[i] == '\t'); @@ -622,7 +621,6 @@ static Dlist *Cache_parse_multiple_fields(const char *header, } i += j; - for ( ; header[i] == ' '; i++); if (header[i] == ':') { /* Field found! */ while (header[++i] == ' ' || header[i] == '\t'); |