diff options
| author | Yousf Ateya <y.ateya@starkbits.com> | 2015-05-10 14:37:54 +0200 |
|---|---|---|
| committer | Yousf Ateya <y.ateya@starkbits.com> | 2015-05-10 16:28:26 +0200 |
| commit | 2ab5d22c0dbf5057977d78b817a98721d59b0f57 (patch) | |
| tree | fa389be9dce42c2b70df5c4e17dc19fe1bb845d2 | |
| parent | 32eb812b28ffc1745e08cb507d8c4409d3ed297a (diff) | |
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
| -rw-r--r-- | res/res_rtp_asterisk.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/res/res_rtp_asterisk.c b/res/res_rtp_asterisk.c index de75626e2..1d43681e4 100644 --- a/res/res_rtp_asterisk.c +++ b/res/res_rtp_asterisk.c @@ -2016,8 +2016,9 @@ static int __rtp_recvfrom(struct ast_rtp_instance *instance, void *buf, size_t s #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; |