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

[Joshua Colp]

Joshua Colp has uploaded a new change for review.

  https://gerrit.asterisk.org/452

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

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

First byte of DTLS 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, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/52/452/1

diff --git a/res/res_rtp_asterisk.c b/res/res_rtp_asterisk.c
index 57b8250..673f940 100644
--- a/res/res_rtp_asterisk.c
+++ b/res/res_rtp_asterisk.c
@@ -1999,8 +1999,9 @@
 #ifdef HAVE_OPENSSL_SRTP
 	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 this is an SSL packet pass it to OpenSSL for processing. RFC section for first byte value:
+	 * https://tools.ietf.org/html/rfc5764#section-5.1.2 */
+	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/452
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iae6fa0d72b37c36a27fe40686e0ae6fba3afec31
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Y Ateya <y.ateya at starkbits.com>