Software /
code /
prosody
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 |