From: Corey Farrell <git@cfware.com>
Date: Tue, 28 Oct 2014 11:20:21 +0000 (+0000)
Subject: app_queue: Cleanup ao2_iterator
X-Git-Tag: 13.1.0-rc1~109
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=688edd55c38f4e89b58b9cc59621885738b512bf;p=thirdparty%2Fasterisk.git

app_queue: Cleanup ao2_iterator

Clean ao2_iterator, resolving reference leak to queue members.

ASTERISK-24454 #close
Reported by: Corey Farrell
Review: https://reviewboard.asterisk.org/r/4111/
........

Merged revisions 426255 from http://svn.asterisk.org/svn/asterisk/branches/11
........

Merged revisions 426260 from http://svn.asterisk.org/svn/asterisk/branches/12


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/13@426266 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---

diff --git a/apps/app_queue.c b/apps/app_queue.c
index 230dcd0478..5190334946 100644
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -3496,6 +3496,7 @@ static void update_realtime_members(struct call_queue *q)
 			}
 			ao2_ref(m, -1);
 		}
+		ao2_iterator_destroy(&mem_iter);
 		ast_debug(3, "Queue %s has no realtime members defined. No need for update\n", q->name);
 		ao2_unlock(q);
 		return;