Merged revisions 336094 via svnmerge from
authorGregory Nietsky <gregory@distrotech.co.za>
Thu, 15 Sep 2011 15:59:24 +0000 (15:59 +0000)
committerGregory Nietsky <gregory@distrotech.co.za>
Thu, 15 Sep 2011 15:59:24 +0000 (15:59 +0000)
commit6f7ff1074b724718ba4ce43ed4de97544000506b
tree5ac8978fc9456fff8d54087ef574da5d046c450e
parent110acf741b182ad4da0770438fcb8075233878f5
Merged revisions 336094 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/10

................
  r336094 | irroot | 2011-09-15 17:54:46 +0200 (Thu, 15 Sep 2011) | 26 lines

  Merged revisions 336093 via svnmerge from
  https://origsvn.digium.com/svn/asterisk/branches/1.8

  ........
    r336093 | irroot | 2011-09-15 17:46:21 +0200 (Thu, 15 Sep 2011) | 20 lines

    Locking order in app_queue.c causes deadlocks.

    a channel lock must never be held with the queues container lock held.

    the deadlock occured on masquerade.

    the queues container lock is a relic of the past the old queue module lock.
    with ao2 there is no need to hold this lock when dealing with members this
    patch removes unneeded locks.

    (closes issue ASTERISK-18101)
    (closes issue ASTERISK-18487)
    Reported by: Paul Rolfe, Jason Legault
    Tested by: irroot, Jason Legault, Paul Rolfe
    Reviewed by: Matthew Nicholson

    Review: https://reviewboard.asterisk.org/r/1402/
  ........
................

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