[asterisk-commits] coreyfarrell: trunk r417213 - in /trunk: ./ main/astobj2_container.c

[SVN commits to the Asterisk project]

Author: coreyfarrell
Date: Wed Jun 25 13:57:04 2014
New Revision: 417213

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=417213
Log:
ao2_container node object ignores REF_DEBUG in all places except one

Almost every reference operation against container node's uses
__ao2_alloc or __ao2_ref, thereby preventing ref logging for
the nodes.  One node reference is released with ao2_t_ref, causing
refcounter.py to falsely report skews and leaks for many nodes.

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

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

Modified:
    trunk/   (props changed)
    trunk/main/astobj2_container.c

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

Modified: trunk/main/astobj2_container.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/astobj2_container.c?view=diff&rev=417213&r1=417212&r2=417213
==============================================================================
--- trunk/main/astobj2_container.c (original)
+++ trunk/main/astobj2_container.c Wed Jun 25 13:57:04 2014
@@ -75,7 +75,8 @@
 	}
 
 	if (flags & AO2_UNLINK_NODE_UNREF_NODE) {
-		ao2_t_ref(node, -1, "Remove node from container");
+		/* Remove node from container */
+		__ao2_ref(node, -1);
 	}
 
 	return 1;