Make the default for the srvlookup option to be yes. It doesn't really make

sense for it to default to off.  The default configuration file has it on, and
proper RFC behavior, as indicated by a comment in the code, is for it to be on.
So, let's have it on by default to make lives easier.
(closes issue #10954, suggested by jtodd)

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@85604 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 37f3e5e3789202e1dbfcbfcfb774608de66b0c57..632b230707fc011625e2bbbbc80935e1050ee09a 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -492,7 +492,7 @@ static const struct cfsip_options {
 #define DEFAULT_NOTIFYMIME     "application/simple-message-summary"
 #define DEFAULT_MWITIME        10
 #define DEFAULT_ALLOWGUEST     TRUE
-#define DEFAULT_SRVLOOKUP      FALSE           /*!< Recommended setting is ON */
+#define DEFAULT_SRVLOOKUP      TRUE            /*!< Recommended setting is ON */
 #define DEFAULT_COMPACTHEADERS FALSE
 #define DEFAULT_TOS_SIP         0               /*!< Call signalling packets should be marked as DSCP CS3, but the default is 0 to be compatible with previous versions. */
 #define DEFAULT_TOS_AUDIO       0               /*!< Audio packets should be marked as DSCP EF (Expedited Forwarding), but the default is 0 to be compatible with previous versions. */
@@ -533,7 +533,7 @@ static int global_rtautoclear;
 static int global_notifyringing;       /*!< Send notifications on ringing */
 static int global_notifyhold;          /*!< Send notifications on hold */
 static int global_alwaysauthreject;    /*!< Send 401 Unauthorized for all failing requests */
-static int srvlookup;                  /*!< SRV Lookup on or off. Default is off, RFC behavior is on */
+static int srvlookup;                  /*!< SRV Lookup on or off. Default is on */
 static int pedanticsipchecking;                /*!< Extra checking ?  Default off */
 static int autocreatepeer;             /*!< Auto creation of peers at registration? Default off. */
 static int global_relaxdtmf;                   /*!< Relax DTMF */