aboutsummaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorKim Alvefur <zash@zash.se>2023-08-23 12:18:34 +0200
committerKim Alvefur <zash@zash.se>2023-08-23 12:18:34 +0200
commit674b91b82b0ea6e3462aa5b26a424925a08fade8 (patch)
tree33c79654e6a8e577c3484f16fbde3f57fa30ec45 /net
parente8128c1d608cb18b1d1e913395fb9b897dd4525e (diff)
downloadprosody-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')
-rw-r--r--net/http/parser.lua8
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;