]> git.ipfire.org Git - thirdparty/asterisk.git/commit
Merged revisions 372288 via svnmerge from
authorAutomerge script <automerge@asterisk.org>
Wed, 5 Sep 2012 14:22:36 +0000 (14:22 +0000)
committerAutomerge script <automerge@asterisk.org>
Wed, 5 Sep 2012 14:22:36 +0000 (14:22 +0000)
commit6ee7c728eefe73f857b1f0dcdd01be479d4875d2
tree5c59166376cf0b651d3ed3bc12b11797016870cf
parente7c0a4dcbb6e1ed80906f0c1b977732db630e078
Merged revisions 372288 via svnmerge from
file:///srv/subversion/repos/asterisk/branches/10

................
  r372288 | mjordan | 2012-09-05 08:42:54 -0500 (Wed, 05 Sep 2012) | 27 lines

  Fix memory leaks in app_voicemail when using IMAP storage or realtime config

  This patch fixes two memory leaks:

  1. When find_user is called with NULL as its first parameter, the voicemail
     user returned is allocated on the heap.  The inboxcount2 function uses
     find_user in such a fashion when counting new messages, and fails to free
     the resulting voicemail user object.

  2. When populate_defaults is called on a voicemail user, it wipes whatever
     flags have been set on the object by copying over the global flags object.
     If the VM_ALLOCED flag was ste on the voicemail user prior to doing so,
     that flag is removed.  This leaks the voicemail user when free_user is later
     called.

  (closes issue ASTERISK-19155)
  Reported by: Filip Jenicek
  patches:
    asterisk.patch2 uploaded by Filip Jenicek (license 6277)

  Patch slightly modified for this commit.

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

  Merged revisions 372268 from http://svn.asterisk.org/svn/asterisk/branches/1.8
................

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/10-digiumphones@372301 65c4cc65-6c06-0410-ace0-fbb531ad65f3
apps/app_voicemail.c