Merged revisions 306356 via svnmerge from
authorJason Parker <jparker@digium.com>
Fri, 4 Feb 2011 19:24:54 +0000 (19:24 +0000)
committerJason Parker <jparker@digium.com>
Fri, 4 Feb 2011 19:24:54 +0000 (19:24 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.8

................
  r306356 | qwell | 2011-02-04 13:24:29 -0600 (Fri, 04 Feb 2011) | 16 lines

  Merged revisions 306346 via svnmerge from
  https://origsvn.digium.com/svn/asterisk/branches/1.6.2

  ........
    r306346 | qwell | 2011-02-04 13:21:43 -0600 (Fri, 04 Feb 2011) | 9 lines

    Don't fallthrough to 'unknown' in the 'ringing' case.

    This could cause improper exits from the queue.

    (closes issue #18499)
    Reported by: zaltar
    Patches:
          app_queue.patch uploaded by zaltar (license 1148)
  ........
................

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

apps/app_queue.c

index 553d57a..2eec835 100644 (file)
@@ -1383,6 +1383,7 @@ static int get_member_status(struct call_queue *q, int max_penalty, int min_pena
                                ast_debug(4, "%s is unavailable because his device state is 'ringing'\n", member->membername);
                                break;
                        }
+                       goto default_case;
                case AST_DEVICE_UNKNOWN:
                        if (conditions & QUEUE_EMPTY_UNKNOWN) {
                                ast_debug(4, "%s is unavailable because his device state is 'unknown'\n", member->membername);