[asterisk-commits] mjordan: branch 11 r374676 - in /branches/11: configs/ res/

[SVN commits to the Asterisk project]

Author: mjordan
Date: Mon Oct  8 15:38:58 2012
New Revision: 374676

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=374676
Log:
Disable ICE support by default

Since there are a number of legacy devices out there that fail to handle ICE
candidates properly (which is a nice way of saying something much uglier),
disable it by default.

Support for ICE candidates can be enabled in rtp.conf using the icesupport
setting.


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

Modified: branches/11/configs/rtp.conf.sample
URL: http://svnview.digium.com/svn/asterisk/branches/11/configs/rtp.conf.sample?view=diff&rev=374676&r1=374675&r2=374676
==============================================================================
--- branches/11/configs/rtp.conf.sample (original)
+++ branches/11/configs/rtp.conf.sample Mon Oct  8 15:38:58 2012
@@ -33,8 +33,8 @@
 ; connected. This option is set to 4 by default.
 ; probation=8
 ;
-; Whether to enable or disable ICE support. This option is enabled by default.
-; icesupport=false
+; Whether to enable or disable ICE support. This option is disabled by default.
+; icesupport=true
 ;
 ; Hostname or address for the STUN server used when determining the external
 ; IP address and port an RTP session can be reached at. The port number is

Modified: branches/11/res/res_rtp_asterisk.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/res/res_rtp_asterisk.c?view=diff&rev=374676&r1=374675&r2=374676
==============================================================================
--- branches/11/res/res_rtp_asterisk.c (original)
+++ branches/11/res/res_rtp_asterisk.c Mon Oct  8 15:38:58 2012
@@ -111,7 +111,7 @@
 };
 
 #define DEFAULT_STRICT_RTP STRICT_RTP_CLOSED
-#define DEFAULT_ICESUPPORT 1
+#define DEFAULT_ICESUPPORT 0
 
 extern struct ast_srtp_res *res_srtp;
 extern struct ast_srtp_policy_res *res_srtp_policy;