diff options
author | David Kalnischkies <david@kalnischkies.de> | 2017-07-24 09:45:51 +0200 |
---|---|---|
committer | David Kalnischkies <david@kalnischkies.de> | 2017-07-26 19:07:56 +0200 |
commit | 1c5f13d489688e5fbbcdd3d0d2dd766769639939 (patch) | |
tree | 3f8760cd7ee2939a20339c3cda6795f4b9bdf2a0 /methods/basehttp.cc | |
parent | 8df85a4fb91bed6c79a3cb9c2000881cc5b42ea7 (diff) |
don't try to parse all fields starting with HTTP as status-line
It is highly unlikely to encounter fields which start with HTTP in
practice, but we should really be a bit more restrictive here.
Diffstat (limited to 'methods/basehttp.cc')
-rw-r--r-- | methods/basehttp.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/methods/basehttp.cc b/methods/basehttp.cc index c3d570c83..47dabf960 100644 --- a/methods/basehttp.cc +++ b/methods/basehttp.cc @@ -85,7 +85,7 @@ bool RequestState::HeaderLine(string const &Line) /*{{{*/ if (Line.empty() == true) return true; - if (Line.size() > 4 && stringcasecmp(Line.data(), Line.data()+4, "HTTP") == 0) + if (Result == 0 && Line.size() > 4 && stringcasecmp(Line.data(), Line.data() + 4, "HTTP") == 0) { // Evil servers return no version if (Line[4] == '/') |