diff options
author | Kim Alvefur <zash@zash.se> | 2015-12-06 02:19:04 +0100 |
---|---|---|
committer | Kim Alvefur <zash@zash.se> | 2015-12-06 02:19:04 +0100 |
commit | b4679e21888856879422f78b57b5740315362d23 (patch) | |
tree | 6c976f1dfe22924ab760c2e51d7ab1bc1b39c6c7 | |
parent | 66b75a0f2ca9d9acbebe12486a50e3341a7c9512 (diff) | |
download | prosody-b4679e21888856879422f78b57b5740315362d23.tar.gz prosody-b4679e21888856879422f78b57b5740315362d23.zip |
mod_blocklist: session[username] can't possibly be unset if that user is sending queries
-rw-r--r-- | plugins/mod_blocklist.lua | 19 |
1 files changed, 9 insertions, 10 deletions
diff --git a/plugins/mod_blocklist.lua b/plugins/mod_blocklist.lua index a870d278..d5e263b1 100644 --- a/plugins/mod_blocklist.lua +++ b/plugins/mod_blocklist.lua @@ -162,7 +162,7 @@ local function edit_blocklist(event) if is_blocking then for jid, in_roster in pairs(new) do - if not blocklist[jid] and in_roster and sessions[username] then + if not blocklist[jid] and in_roster then for _, session in pairs(sessions[username].sessions) do if session.presence then module:send(st.presence({ type = "unavailable", to = jid, from = session.full_jid })); @@ -171,15 +171,14 @@ local function edit_blocklist(event) end end end - if sessions[username] then - local blocklist_push = st.iq({ type = "set", id = "blocklist-push" }) - :add_child(action); -- I am lazy - - for _, session in pairs(sessions[username].sessions) do - if session.interested_blocklist then - blocklist_push.attr.to = session.full_jid; - session.send(blocklist_push); - end + + local blocklist_push = st.iq({ type = "set", id = "blocklist-push" }) + :add_child(action); -- I am lazy + + for _, session in pairs(sessions[username].sessions) do + if session.interested_blocklist then + blocklist_push.attr.to = session.full_jid; + session.send(blocklist_push); end end |