Make a minor tweak to admin_exec() - don't lock the conference list until it

is actually necessary.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@63326 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/apps/app_meetme.c b/apps/app_meetme.c
index 01cf838a654ca98a24767e358f77e8d5d1ed9b98..0e50414db8f3769bdd9d57ca32f2b6952f549927 100644 (file)
--- a/apps/app_meetme.c
+++ b/apps/app_meetme.c
@@ -2768,17 +2768,16 @@ static int admin_exec(struct ast_channel *chan, void *data) {
 
        u = ast_module_user_add(chan);
 
-       AST_LIST_LOCK(&confs);
-       
        params = ast_strdupa(data);
        AST_STANDARD_APP_ARGS(args, params);
 
        if (!args.command) {
                ast_log(LOG_WARNING, "MeetmeAdmin requires a command!\n");
-               AST_LIST_UNLOCK(&confs);
                ast_module_user_remove(u);
                return -1;
        }
+
+       AST_LIST_LOCK(&confs);
        AST_LIST_TRAVERSE(&confs, cnf, list) {
                if (!strcmp(cnf->confno, args.confno))
                        break;
@@ -2953,7 +2952,7 @@ static int channel_admin_exec(struct ast_channel *chan, void *data) {
        }
        
        /* perform the specified action */
-       switch(*args.command) {
+       switch (*args.command) {
                case 77: /* M: Mute */ 
                        user->adminflags |= ADMINFLAG_MUTED;
                        break;