diff options
| author | jpeeler <jpeeler@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-07-01 20:19:16 +0000 |
|---|---|---|
| committer | jpeeler <jpeeler@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-07-01 20:19:16 +0000 |
| commit | df3e95c62ff9cd3d57879bc213260d0bf08df5b3 (patch) | |
| tree | 5811614ed930a5794105b6ec10f1bcaf53e1cf51 /apps | |
| parent | 635567d558089cc9c977f13beaec841bbc4b9ddc (diff) | |
Allow admin user to join conference without using admin mode and no user pin.
Configuring the conference in meetme.conf like the following:
conf => 2345,,6666
did not prompt for pin when used without admin mode. This meant that the
conference could not be joined as an admin even if the user knew the correct
pin. The original bug report was submitted claiming that the blank user pin
should deny entry into the conference. I think a better way to handle this
would be with a feature enhancement that used the following syntax:
conf => 2345,X,6666 - where X denotes no acceptable pin allowed
(closes issue #15704)
Reported by: modelnine
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@273474 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/app_meetme.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/apps/app_meetme.c b/apps/app_meetme.c index a13844e9a..c25d8929d 100644 --- a/apps/app_meetme.c +++ b/apps/app_meetme.c @@ -2889,8 +2889,7 @@ static int conf_exec(struct ast_channel *chan, void *data) } else { if (((!ast_strlen_zero(cnf->pin) && !ast_test_flag(&confflags, CONFFLAG_ADMIN)) || - (!ast_strlen_zero(cnf->pinadmin) && - ast_test_flag(&confflags, CONFFLAG_ADMIN))) && + !ast_strlen_zero(cnf->pinadmin)) && (!(cnf->users == 0 && cnf->isdynamic))) { char pin[MAX_PIN] = ""; int j; |