From bceb0845e63858b87259366ec1d0680e051d4d4b Mon Sep 17 00:00:00 2001 From: Mark Michelson <mmichelson@digium.com> Date: Tue, 21 Oct 2008 23:08:56 +0000 Subject: [PATCH] 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 --- channels/chan_sip.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) 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; } -- GitLab