Comparison

plugins/mod_pep_plus.lua @ 9061:82dd435c942c

mod_pep_plus: Fix boolean logic precedence (thanks Link Mauve) In case of 'from' subscription to a JID without node, eg a component, the condition becomes true and update_subscriptions() is called with contact_node = nil, which causes a traceback.
author Kim Alvefur <zash@zash.se>
date Sun, 29 Jul 2018 18:34:38 +0200
parent 9030:7d0b7e086c6a
child 9074:0462405b1cfb
comparison
equal deleted inserted replaced
9060:69bc3144c2b7 9061:82dd435c942c
421 if is_self then 421 if is_self then
422 -- Optimization: Fiddle with other local users 422 -- Optimization: Fiddle with other local users
423 for jid, item in pairs(origin.roster) do -- for all interested contacts 423 for jid, item in pairs(origin.roster) do -- for all interested contacts
424 if jid then 424 if jid then
425 local contact_node, contact_host = jid_split(jid); 425 local contact_node, contact_host = jid_split(jid);
426 if contact_host == host and item.subscription == "both" or item.subscription == "from" then 426 if contact_host == host and (item.subscription == "both" or item.subscription == "from") then
427 update_subscriptions(user_bare, contact_node, notify); 427 update_subscriptions(user_bare, contact_node, notify);
428 end 428 end
429 end 429 end
430 end 430 end
431 end 431 end