diff options
| author | Jonathan Rose <jrose@digium.com> | 2012-07-05 18:11:58 +0000 |
|---|---|---|
| committer | Jonathan Rose <jrose@digium.com> | 2012-07-05 18:11:58 +0000 |
| commit | 70e34d335437db2624ee1adfdfcc0565a9a93ea0 (patch) | |
| tree | 294d12378eb3570bfe49abbdc4947f7428952ddd /apps/app_mixmonitor.c | |
| parent | 4b3476d01627511362c2a783ad228ca0c3e85993 (diff) | |
app_mixmonitor: Fix a reference leak in manager_mixmonitor function
Manager_mixmonitor included an early return on failed executions of mixmonitor
that would result in a leaked channel reference.
(closes issue ASTERISK-19943)
Reported by: Mark Murawski
Patches:
mixmonitor-trunk-368394.patch uploaded by Mark Murawski (license 5791)
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@369644 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'apps/app_mixmonitor.c')
| -rw-r--r-- | apps/app_mixmonitor.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/apps/app_mixmonitor.c b/apps/app_mixmonitor.c index 3a72380d6..b3c37f31e 100644 --- a/apps/app_mixmonitor.c +++ b/apps/app_mixmonitor.c @@ -1258,6 +1258,7 @@ static int manager_mixmonitor(struct mansession *s, const struct message *m) ast_channel_unlock(c); if (res) { + c = ast_channel_unref(c); astman_send_error(s, m, "Could not start monitoring channel"); return AMI_SUCCESS; } |