From fac74808203c697677a87d34ccc409a3fe203a95 Mon Sep 17 00:00:00 2001 From: Russell Bryant Date: Fri, 30 Nov 2007 19:34:47 +0000 Subject: Merged revisions 90348 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r90348 | russell | 2007-11-30 13:26:04 -0600 (Fri, 30 Nov 2007) | 8 lines Change the behavior of ao2_link(). Previously, in inherited a reference. Now, it automatically increases the reference count to reflect the reference that is now held by the container. This was done to be more consistent with ao2_unlink(), which automatically releases the reference held by the container. It also makes it so it is no longer possible for a pointer to be invalid after ao2_link() returns. ........ git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@90351 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- main/astobj2.c | 1 + 1 file changed, 1 insertion(+) (limited to 'main/astobj2.c') diff --git a/main/astobj2.c b/main/astobj2.c index e7a091c80..2d3611889 100644 --- a/main/astobj2.c +++ b/main/astobj2.c @@ -363,6 +363,7 @@ void *ao2_link(struct ao2_container *c, void *user_data) p->version = ast_atomic_fetchadd_int(&c->version, 1); AST_LIST_INSERT_TAIL(&c->buckets[i], p, entry); ast_atomic_fetchadd_int(&c->elements, 1); + ao2_ref(user_data, +1); ao2_unlock(c); return p; -- cgit v1.2.3