diff options
author | Kim Alvefur <zash@zash.se> | 2023-08-23 12:18:34 +0200 |
---|---|---|
committer | Kim Alvefur <zash@zash.se> | 2023-08-23 12:18:34 +0200 |
commit | 674b91b82b0ea6e3462aa5b26a424925a08fade8 (patch) | |
tree | 33c79654e6a8e577c3484f16fbde3f57fa30ec45 /net/http | |
parent | e8128c1d608cb18b1d1e913395fb9b897dd4525e (diff) | |
download | prosody-674b91b82b0ea6e3462aa5b26a424925a08fade8.tar.gz prosody-674b91b82b0ea6e3462aa5b26a424925a08fade8.zip |
net.http.parser: Reject overlarge header section earlier
This case would eventually be rejected by the buffer size limit.
Diffstat (limited to 'net/http')
-rw-r--r-- | net/http/parser.lua | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/net/http/parser.lua b/net/http/parser.lua index a6624662..12d40883 100644 --- a/net/http/parser.lua +++ b/net/http/parser.lua @@ -59,7 +59,13 @@ function httpstream.new(success_cb, error_cb, parser_type, options_cb) while buffer:length() > 0 do if state == nil then -- read request local index = buffer:sub(1, headlimit):find("\r\n\r\n", nil, true); - if not index then return; end -- not enough data + if not index then + if buffer:length() > headlimit then + return error_cb("header-too-large"); + end + -- not enough data + return; + end -- FIXME was reason_phrase meant to be passed on somewhere? local method, path, httpversion, status_code, reason_phrase; -- luacheck: ignore reason_phrase local first_line; |