From: Holger Weiss <holger@zedat.fu-berlin.de>
Date: Wed, 24 Jul 2019 08:45:30 +0000 (+0200)
Subject: mod_privacy: Don't try to look up 'undefined' list
X-Git-Tag: 19.08~40
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=4218aecd1c7f58207ab17d3cccf2886d5558d62b;p=ejabberd

mod_privacy: Don't try to look up 'undefined' list

Don't crash if the <query/> element of an IQ of type 'set' contains
neither a <default/> nor an <active/> child.

This fixes a regression introduced in commit
8410a203ec81f0b0b9e4885221abf1d7ba6bd9f1.
---

diff --git a/src/mod_privacy.erl b/src/mod_privacy.erl
index f928c8291..d54e79f0b 100644
--- a/src/mod_privacy.erl
+++ b/src/mod_privacy.erl
@@ -413,6 +413,8 @@ update_c2s_state_with_privacy_list(#iq{type = set,
 				   State) ->
     %% Match a IQ set containing a new active privacy list
     case xmpp:get_subtag(IQ, #privacy_query{}) of
+	#privacy_query{active = undefined} ->
+	    State;
 	#privacy_query{default = undefined, active = Active} ->
 	    case Active of
 		none ->