[Asterisk-code-review] chan ooh323: call ast rtp instance stop on ooh323 destroy (asterisk[13])

Joshua Colp asteriskteam at digium.com
Tue Sep 8 07:29:17 CDT 2015


Joshua Colp has submitted this change and it was merged.

Change subject: chan_ooh323: call ast_rtp_instance_stop on ooh323_destroy
......................................................................


chan_ooh323: call ast_rtp_instance_stop on ooh323_destroy

    Call ast_rtp_instance_stop on ooh323_destroy to free resources
    allocated by rtp instance

    ASTERISK-25299 #close
    Report by: Alexandr Dranchuk

Change-Id: I455096bd7da016b871afe90af86067c2c7c9f33f
---
M addons/chan_ooh323.c
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Anonymous Coward #1000019: Verified
  Matt Jordan: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved



diff --git a/addons/chan_ooh323.c b/addons/chan_ooh323.c
index 6268ad0..ef67b78 100644
--- a/addons/chan_ooh323.c
+++ b/addons/chan_ooh323.c
@@ -4115,6 +4115,7 @@
 		}
 
 		if (cur->rtp) {
+			ast_rtp_instance_stop(cur->rtp);
 			ast_rtp_instance_destroy(cur->rtp);
 			cur->rtp = NULL;
 		}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I455096bd7da016b871afe90af86067c2c7c9f33f
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Alexander Anikin <may213 at yandex.ru>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list