[asterisk-commits] coreyfarrell: branch 12 r426260 - in /branches/12: ./ apps/app_queue.c

[SVN commits to the Asterisk project]

Author: coreyfarrell
Date: Tue Oct 28 06:19:02 2014
New Revision: 426260

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=426260
Log:
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

Modified:
    branches/12/   (props changed)
    branches/12/apps/app_queue.c

Propchange: branches/12/
------------------------------------------------------------------------------
Binary property 'branch-11-merged' - no diff available.

Modified: branches/12/apps/app_queue.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/apps/app_queue.c?view=diff&rev=426260&r1=426259&r2=426260
==============================================================================
--- branches/12/apps/app_queue.c (original)
+++ branches/12/apps/app_queue.c Tue Oct 28 06:19:02 2014
@@ -3396,6 +3396,7 @@
 			}
 			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;