[asterisk-commits] oej: branch 10 r337178 - in /branches/10: ./ configs/ res/

[SVN commits to the Asterisk project]

Author: oej
Date: Wed Sep 21 03:51:41 2011
New Revision: 337178

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=337178
Log:
Change strictrtp option to default to yes in the RTP module

Suggested by Kapejod on Facebook

Review: https://reviewboard.asterisk.org/r/1448/
(closes issue ASTERISK-18587)

Thanks for quick feedback to kpfleming and Tilghman
--Denna och nedanstående rader kommer inte med i loggmeddelandet--

M    CHANGES
M    configs/rtp.conf.sample
M    res/res_rtp_asterisk.c

Modified:
    branches/10/CHANGES
    branches/10/configs/rtp.conf.sample
    branches/10/res/res_rtp_asterisk.c

Modified: branches/10/CHANGES
URL: http://svnview.digium.com/svn/asterisk/branches/10/CHANGES?view=diff&rev=337178&r1=337177&r2=337178
==============================================================================
--- branches/10/CHANGES (original)
+++ branches/10/CHANGES Wed Sep 21 03:51:41 2011
@@ -222,6 +222,11 @@
 ------------
 * authdebug is now disabled by default. To enable this functionaility again
    set authdebug = yes in iax.conf.
+
+RTP Changes
+-----------
+ * The rtp.conf setting "strictrtp" is now enabled by default. In previous
+   releases it was disabled.
 
 ------------------------------------------------------------------------------
 --- Functionality changes from Asterisk 1.6.2 to Asterisk 1.8 ----------------

Modified: branches/10/configs/rtp.conf.sample
URL: http://svnview.digium.com/svn/asterisk/branches/10/configs/rtp.conf.sample?view=diff&rev=337178&r1=337177&r2=337178
==============================================================================
--- branches/10/configs/rtp.conf.sample (original)
+++ branches/10/configs/rtp.conf.sample Wed Sep 21 03:51:41 2011
@@ -23,5 +23,5 @@
 ;
 ; Enable strict RTP protection. This will drop RTP packets that
 ; do not come from the source of the RTP stream. This option is
-; disabled by default.
+; enabled by default.
 ; strictrtp=yes

Modified: branches/10/res/res_rtp_asterisk.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/res/res_rtp_asterisk.c?view=diff&rev=337178&r1=337177&r2=337178
==============================================================================
--- branches/10/res/res_rtp_asterisk.c (original)
+++ branches/10/res/res_rtp_asterisk.c Wed Sep 21 03:51:41 2011
@@ -2829,7 +2829,7 @@
 	rtpstart = DEFAULT_RTP_START;
 	rtpend = DEFAULT_RTP_END;
 	dtmftimeout = DEFAULT_DTMF_TIMEOUT;
-	strictrtp = STRICT_RTP_OPEN;
+	strictrtp = STRICT_RTP_CLOSED;
 	if (cfg) {
 		if ((s = ast_variable_retrieve(cfg, "general", "rtpstart"))) {
 			rtpstart = atoi(s);