Merged revisions 334357 via svnmerge from
authorRichard Mudgett <rmudgett@digium.com>
Fri, 2 Sep 2011 21:09:31 +0000 (21:09 +0000)
committerRichard Mudgett <rmudgett@digium.com>
Fri, 2 Sep 2011 21:09:31 +0000 (21:09 +0000)
commit35e27201c7b942396e01fcabf4aeba2edfe9f544
tree652f71093d337841ac2c6ac63274b815ec402f2a
parent220bf145570d6ad584c174fc531793d699976400
Merged revisions 334357 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/10

................
  r334357 | rmudgett | 2011-09-02 16:08:16 -0500 (Fri, 02 Sep 2011) | 26 lines

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

  ........
    r334355 | rmudgett | 2011-09-02 15:59:49 -0500 (Fri, 02 Sep 2011) | 19 lines

    MusicOnHold has extra unref which may lead to memory corruption and crash.

    The problem happens when a call is disconnected and you had started a MOH
    class that does not use the files mode.  If you define REF_DEBUG and
    recreate the problem, it will announce itself with the following warning:
    Attempt to unref mohclass 0xb70722e0 (default) when only 1 ref remained,
    and class is still in a container!

    * Fixed moh_alloc() and moh_release() functions not handling the
    state->class reference consistently.

    (closes issue ASTERISK-18346)
    Reported by: Mark Murawski
    Patches:
          jira_asterisk_18346_v1.8.patch (license #5621) patch uploaded by rmudgett
    Tested by: rmudgett, Mark Murawski

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

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