diff options
| author | Jonathan Rose <jrose@digium.com> | 2012-11-28 16:47:44 +0000 |
|---|---|---|
| committer | Jonathan Rose <jrose@digium.com> | 2012-11-28 16:47:44 +0000 |
| commit | 89e84ff75d7100f675c141d9d0e1bbef02a0fdcd (patch) | |
| tree | 55311f8832eb584ba4af9dbdcd189306da391e0a /main/manager.c | |
| parent | 95a4a82702e27826f48f3ef8dfb63ea518b19f23 (diff) | |
manager: Make challenge work with allowmultiplelogin=no
Prior to this patch, challenge would yield a multiple logins error if used
without providing the username (which isn't really supposed to be an argument
to challenge) if allowmultiplelogin was set to no because allowmultiplelogin
finds a user with a zero length login name. This check is simply disabled for
the challenge action when the username is empty by this patch.
(closes issue ASTERISK-20677)
Reported by: Vladimir
Patches:
challenge_action_nomultiplelogin.diff uploaded by Jonathan Rose (license 6182)
........
Merged revisions 376725 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 376726 from http://svn.asterisk.org/svn/asterisk/branches/10
........
Merged revisions 376727 from http://svn.asterisk.org/svn/asterisk/branches/11
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@376728 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'main/manager.c')
| -rw-r--r-- | main/manager.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/main/manager.c b/main/manager.c index d08982571..011ae103e 100644 --- a/main/manager.c +++ b/main/manager.c @@ -5127,7 +5127,7 @@ static int process_message(struct mansession *s, const struct message *m) || !strcasecmp(action, "Challenge"))) { user = astman_get_header(m, "Username"); - if (check_manager_session_inuse(user)) { + if (!ast_strlen_zero(user) && check_manager_session_inuse(user)) { report_session_limit(s); sleep(1); mansession_lock(s); |