* src/mod_muc/mod_muc_room.erl: Owner of a password protected room

must provide the password, like other participants (EJAB-867)

SVN Revision: 1880

diff --git a/ChangeLog b/ChangeLog
index 9e05d21252abfbf1ac38977a3b3c95189231b1d1..b054ec9b2b2598117813a310ebe9735500cb5b8b 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2009-02-16  Badlop  <badlop@process-one.net>
 
+       * src/mod_muc/mod_muc_room.erl: Owner of a password protected room
+       must provide the password, like other participants (EJAB-867)
+
        * src/mod_muc/mod_muc_log.erl: Prevent XSS in MUC logs by
        linkifying only a few known protocols (EJAB-850)
 

diff --git a/src/mod_muc/mod_muc_room.erl b/src/mod_muc/mod_muc_room.erl
index 16de90cbfa3e31f9411b848e889a8d45ed4e47c3..34a602585b4bcf2b8285fc6ae67e95b6614ba099 100644 (file)
--- a/src/mod_muc/mod_muc_room.erl
+++ b/src/mod_muc/mod_muc_room.erl
@@ -1489,7 +1489,7 @@ add_new_user(From, Nick, {xmlelement, _, Attrs, Els} = Packet, StateData) ->
              From, Err),
            StateData;
        {_, _, _, Role} ->
-           case check_password(Affiliation, Els, StateData) of
+           case check_password(ServiceAffiliation, Els, StateData) of
                true ->
                    NewState =
                        add_user_presence(
@@ -1546,8 +1546,9 @@ add_new_user(From, Nick, {xmlelement, _, Attrs, Els} = Packet, StateData) ->
     end.
 
 check_password(owner, _Els, _StateData) ->
+    %% Don't check pass if user is owner in MUC service (access_admin option)
     true;
-check_password(_Affiliation, Els, StateData) ->
+check_password(_ServiceAffiliation, Els, StateData) ->
     case (StateData#state.config)#config.password_protected of
        false ->
            true;