From 70e34d335437db2624ee1adfdfcc0565a9a93ea0 Mon Sep 17 00:00:00 2001 From: Jonathan Rose Date: Thu, 5 Jul 2012 18:11:58 +0000 Subject: 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 --- apps/app_mixmonitor.c | 1 + 1 file changed, 1 insertion(+) (limited to 'apps') 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; } -- cgit v1.2.3