aboutsummaryrefslogtreecommitdiffstats
path: root/lib/inets/src/http_server
diff options
context:
space:
mode:
authorIngela Anderton Andin <[email protected]>2018-04-12 14:24:08 +0200
committerIngela Anderton Andin <[email protected]>2018-05-22 18:07:50 +0200
commitdaba7e0abe4a5642543676e966298b08dee83eb9 (patch)
tree244a35a9694091b697b257a1fe915cca89fcf116 /lib/inets/src/http_server
parent40ab3958644be3fc14682c54233514fdf880386a (diff)
downloadotp-daba7e0abe4a5642543676e966298b08dee83eb9.tar.gz
otp-daba7e0abe4a5642543676e966298b08dee83eb9.tar.bz2
otp-daba7e0abe4a5642543676e966298b08dee83eb9.zip
inets: Gracefully handle bad headers
max_headers operated on the individual header length instead of the total length of all headers. Also headers with empty keys are now discarded.
Diffstat (limited to 'lib/inets/src/http_server')
-rw-r--r--lib/inets/src/http_server/httpd_request.erl6
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/inets/src/http_server/httpd_request.erl b/lib/inets/src/http_server/httpd_request.erl
index 007d272323..e513eb8a3a 100644
--- a/lib/inets/src/http_server/httpd_request.erl
+++ b/lib/inets/src/http_server/httpd_request.erl
@@ -259,17 +259,17 @@ parse_headers(<<?LF, Octet, Rest/binary>>, Header, Headers, Current, Max,
%% If ?CR is is missing RFC2616 section-19.3
parse_headers(<<?CR,?LF, Octet, Rest/binary>>, Header, Headers, Current, Max,
Options, Result);
-parse_headers(<<?CR,?LF, Octet, Rest/binary>>, Header, Headers, _, Max,
+parse_headers(<<?CR,?LF, Octet, Rest/binary>>, Header, Headers, Current, Max,
Options, Result) ->
case http_request:key_value(lists:reverse(Header)) of
undefined -> %% Skip headers with missing :
parse_headers(Rest, [Octet], Headers,
- 0, Max, Options, Result);
+ Current, Max, Options, Result);
NewHeader ->
case check_header(NewHeader, Options) of
ok ->
parse_headers(Rest, [Octet], [NewHeader | Headers],
- 0, Max, Options, Result);
+ Current, Max, Options, Result);
{error, Reason} ->
HttpVersion = lists:nth(3, lists:reverse(Result)),
{error, Reason, HttpVersion}