diff options
author | Matthew Wild <mwild1@gmail.com> | 2013-04-22 12:24:42 +0100 |
---|---|---|
committer | Matthew Wild <mwild1@gmail.com> | 2013-04-22 12:24:42 +0100 |
commit | eb8fe8a1a76951986e981bb592782a79ef5a09ba (patch) | |
tree | a3ceaf9ce849b643d85d0c0b600755225e07df55 | |
parent | 71120cf0227edd516932f9037da61e70f47a3d07 (diff) | |
download | prosody-eb8fe8a1a76951986e981bb592782a79ef5a09ba.tar.gz prosody-eb8fe8a1a76951986e981bb592782a79ef5a09ba.zip |
net.server.http: Ensure that event map cannot grow forever (limit to 10K wildcard-only entries)
-rw-r--r-- | net/http/server.lua | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/net/http/server.lua b/net/http/server.lua index a983b8d5..3c2b55d5 100644 --- a/net/http/server.lua +++ b/net/http/server.lua @@ -27,6 +27,8 @@ local function is_wildcard_match(wildcard_event, event) return wildcard_event:sub(1, -2) == event:sub(1, #wildcard_event-1); end +local recent_wildcard_events, max_cached_wildcard_events = {}, 10000; + local event_map = events._event_map; setmetatable(events._handlers, { __index = function (handlers, curr_event) @@ -58,6 +60,12 @@ setmetatable(events._handlers, { handlers_array = false; end rawset(handlers, curr_event, handlers_array); + if not event_map[curr_event] then -- Only wildcard handlers match, if any + table.insert(recent_wildcard_events, curr_event); + if #recent_wildcard_events > max_cached_wildcard_events then + rawset(handlers, table.remove(recent_wildcard_events, 1), nil); + end + end return handlers_array; end; __newindex = function (handlers, curr_event, handlers_array) |