[Asterisk-code-review] rtp engine: Add missing unlock. (asterisk[15])

Joshua Colp asteriskteam at digium.com
Thu Jan 4 14:57:34 CST 2018


Joshua Colp has submitted this change and it was merged. ( https://gerrit.asterisk.org/7803 )

Change subject: rtp_engine: Add missing unlock.
......................................................................

rtp_engine: Add missing unlock.

Change-Id: I380c31a255e060309f4916da11176e0d00813215
---
M main/rtp_engine.c
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Kevin Harwell: Looks good to me, but someone else must approve
  Benjamin Keith Ford: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved; Approved for Submit



diff --git a/main/rtp_engine.c b/main/rtp_engine.c
index 76bdf87..764be19 100644
--- a/main/rtp_engine.c
+++ b/main/rtp_engine.c
@@ -2922,6 +2922,7 @@
 		if (payload < 0) {
 			ast_log(LOG_WARNING, "No dynamic RTP payload type values available "
 				"for %s - %d!\n", format ? ast_format_get_name(format) : "", rtp_code);
+			ast_rwlock_unlock(&static_RTP_PT_lock);
 			return;
 		}
 	}

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

Gerrit-Project: asterisk
Gerrit-Branch: 15
Gerrit-MessageType: merged
Gerrit-Change-Id: I380c31a255e060309f4916da11176e0d00813215
Gerrit-Change-Number: 7803
Gerrit-PatchSet: 1
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180104/d55c02b1/attachment-0001.html>


More information about the asterisk-code-review mailing list