[Asterisk-code-review] res rtp asterisk: Correction for the limit which detects tha... (asterisk[master])

Y Ateya asteriskteam at digium.com
Sun May 10 07:41:43 CDT 2015


Y Ateya has uploaded a new change for review.

  https://gerrit.asterisk.org/428

Change subject: res_rtp_asterisk: Correction for the limit which detects that a packet is SRTP.
......................................................................

res_rtp_asterisk: Correction for the limit which detects that a packet is SRTP.

First byte of SRTP packet shall be in range 20-63, not 20-64. Refer to RFC
https://tools.ietf.org/html/rfc5764#section-5.1.2 for correct values.

Change-Id: Iae6fa0d72b37c36a27fe40686e0ae6fba3afec31
---
M res/res_rtp_asterisk.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/28/428/1

diff --git a/res/res_rtp_asterisk.c b/res/res_rtp_asterisk.c
index de75626..10abb59 100644
--- a/res/res_rtp_asterisk.c
+++ b/res/res_rtp_asterisk.c
@@ -2017,7 +2017,7 @@
 	dtls_srtp_check_pending(instance, rtp, rtcp);
 
 	/* If this is an SSL packet pass it to OpenSSL for processing */
-	if ((*in >= 20) && (*in <= 64)) {
+	if ((*in >= 20) && (*in <= 63)) {
 		struct dtls_details *dtls = !rtcp ? &rtp->dtls : &rtp->rtcp->dtls;
 		int res = 0;
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iae6fa0d72b37c36a27fe40686e0ae6fba3afec31
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Y Ateya <y.ateya at starkbits.com>



More information about the asterisk-code-review mailing list