[Asterisk-code-review] res xmpp: Fix ref counting issue (asterisk[13])

Sean Bright asteriskteam at digium.com
Thu Mar 23 09:54:42 CDT 2017


Sean Bright has uploaded a new change for review. ( https://gerrit.asterisk.org/5287 )

Change subject: res_xmpp: Fix ref counting issue
......................................................................

res_xmpp: Fix ref counting issue

The only remaining reference to the endpoint is in the endpoints
container, and because it is unlinked in ast_endpoint_shutdown, we don't
have to explicitly cleanup the endpoint ourselves.

Change-Id: I912a2692e52d3e2ed445b32d8ae3f9004bc2f2e8
---
M res/res_xmpp.c
1 file changed, 0 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/87/5287/1

diff --git a/res/res_xmpp.c b/res/res_xmpp.c
index 5d7fe66..eb48d12 100644
--- a/res/res_xmpp.c
+++ b/res/res_xmpp.c
@@ -561,7 +561,6 @@
 	ast_xmpp_client_disconnect(client);
 
 	ast_endpoint_shutdown(client->endpoint);
-	ao2_cleanup(client->endpoint);
 	client->endpoint = NULL;
 
 	if (client->filter) {

-- 
To view, visit https://gerrit.asterisk.org/5287
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I912a2692e52d3e2ed445b32d8ae3f9004bc2f2e8
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>



More information about the asterisk-code-review mailing list