[Asterisk-code-review] channel: Fix leak on successful call to chan->tech->requester. (asterisk[master])

Corey Farrell asteriskteam at digium.com
Sat Aug 5 14:46:01 CDT 2017


Corey Farrell has uploaded this change for review. ( https://gerrit.asterisk.org/6169


Change subject: channel: Fix leak on successful call to chan->tech->requester.
......................................................................

channel: Fix leak on successful call to chan->tech->requester.

joint_cap needs to be released unconditionally as chan->tech->requester
does not steal the reference even on success.

ASTERISK-27180 #close

Change-Id: I647728992559bdb0a9c7357c20be1b36400d68b6
---
M main/channel.c
1 file changed, 3 insertions(+), 8 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/69/6169/1

diff --git a/main/channel.c b/main/channel.c
index 6682555..e6a1752 100644
--- a/main/channel.c
+++ b/main/channel.c
@@ -6063,9 +6063,6 @@
 		c = chan->tech->requester_with_stream_topology(type, topology, assignedids, requestor, addr, cause);
 
 		ast_stream_topology_free(tmp_converted_topology);
-		if (!c) {
-			return NULL;
-		}
 	} else if (chan->tech->requester) {
 		struct ast_format_cap *tmp_converted_cap = NULL;
 		struct ast_format_cap *tmp_cap;
@@ -6116,12 +6113,10 @@
 		ao2_cleanup(tmp_converted_cap);
 
 		c = chan->tech->requester(type, joint_cap, assignedids, requestor, addr, cause);
+		ao2_ref(joint_cap, -1);
+	}
 
-		if (!c) {
-			ao2_ref(joint_cap, -1);
-			return NULL;
-		}
-	} else {
+	if (!c) {
 		return NULL;
 	}
 

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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I647728992559bdb0a9c7357c20be1b36400d68b6
Gerrit-Change-Number: 6169
Gerrit-PatchSet: 1
Gerrit-Owner: Corey Farrell <git at cfware.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20170805/ae96e0c5/attachment.html>


More information about the asterisk-code-review mailing list