From: Mark Michelson <mmichelson@digium.com>
Date: Tue, 21 Oct 2008 23:08:56 +0000 (+0000)
Subject: When attempting to resolve hostnames, we need to be sure
X-Git-Tag: 1.6.2.0-beta1~1050
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=bceb0845e63858b87259366ec1d0680e051d4d4b;p=thirdparty%2Fasterisk.git

When attempting to resolve hostnames, we need to be sure
to remove any parameters from the string so that name
resolution succeeds.

(closes issue #13727)
Reported by: fnordian
Patches:
      resolvewithouturiparameter.patch uploaded by fnordian (license 110)



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

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 1349480e69..1df8b72740 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -21927,9 +21927,17 @@ static struct sip_peer *build_peer(const char *name, struct ast_variable *v, str
 
 	if (srvlookup && peer->dnsmgr == NULL) {
 		char transport[MAXHOSTNAMELEN];
+		char _srvlookup[MAXHOSTNAMELEN];
+		char *params;
+
+		ast_copy_string(_srvlookup, srvlookup, sizeof(_srvlookup));
+		if (params = strchr(_srvlookup, ';')) {
+			*params++ = '\0';
+		}
+		
 		snprintf(transport, sizeof(transport), "_sip._%s", get_transport(peer->socket.type));
 
-		if (ast_dnsmgr_lookup(srvlookup, &peer->addr, &peer->dnsmgr, global_srvlookup ? transport : NULL)) {
+		if (ast_dnsmgr_lookup(_srvlookup, &peer->addr, &peer->dnsmgr, global_srvlookup ? transport : NULL)) {
 			unref_peer(peer, "getting rid of a peer pointer");
 			return NULL;
 		}