summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorcorvid <corvid@lavabit.com>2010-02-25 21:15:45 +0000
committercorvid <corvid@lavabit.com>2010-02-25 21:15:45 +0000
commitae6386f8c9def1c247b99902d1315ceb5c20b31b (patch)
tree7e557acb4d339d3852f72fca597cdfafecec3f50
parent0bd335403e7793551bb76cbdf629fbba05fdfecc (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."
-rw-r--r--src/cache.c2
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');