[Asterisk-code-review] bridge softmix.c: Fix memory leak. (asterisk[master])

Jenkins2 asteriskteam at digium.com
Fri Jun 22 08:23:59 CDT 2018


Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/9272 )

Change subject: bridge_softmix.c: Fix memory leak.
......................................................................

bridge_softmix.c: Fix memory leak.

Made release the media_types vector in
softmix_bridge_stream_topology_changed().

Change-Id: Ide3f47e379b614879220dfeeb13843f9f2b177b5
---
M bridges/bridge_softmix.c
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Corey Farrell: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Jenkins2: Approved for Submit



diff --git a/bridges/bridge_softmix.c b/bridges/bridge_softmix.c
index 46b27f1..e04e1fa 100644
--- a/bridges/bridge_softmix.c
+++ b/bridges/bridge_softmix.c
@@ -2218,6 +2218,8 @@
 		ast_channel_unlock(participant->chan);
 		ast_bridge_channel_unlock(participant);
 	}
+
+	AST_VECTOR_FREE(&media_types);
 }
 
 static struct ast_bridge_technology softmix_bridge = {

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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ide3f47e379b614879220dfeeb13843f9f2b177b5
Gerrit-Change-Number: 9272
Gerrit-PatchSet: 1
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180622/51579395/attachment.html>


More information about the asterisk-code-review mailing list