mirror of
				https://github.com/asterisk/asterisk.git
				synced 2025-10-31 10:47:18 +00:00 
			
		
		
		
	Don't do a DNS lookup on an outbound REGISTER host if there is an outbound proxy configured.
(closes issue ASTERISK-16550) reported by: Olle Johansson git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@352424 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
		| @@ -13069,7 +13069,14 @@ static int transmit_register(struct sip_registry *r, int sipmethod, const char * | |||||||
| 		peer = find_peer(r->hostname, NULL, TRUE, FINDPEERS, FALSE, 0); | 		peer = find_peer(r->hostname, NULL, TRUE, FINDPEERS, FALSE, 0); | ||||||
| 		snprintf(transport, sizeof(transport), "_%s._%s",get_srv_service(r->transport), get_srv_protocol(r->transport)); /* have to use static get_transport function */ | 		snprintf(transport, sizeof(transport), "_%s._%s",get_srv_service(r->transport), get_srv_protocol(r->transport)); /* have to use static get_transport function */ | ||||||
| 		r->us.ss.ss_family = get_address_family_filter(&bindaddr); /* Filter address family */ | 		r->us.ss.ss_family = get_address_family_filter(&bindaddr); /* Filter address family */ | ||||||
| 		ast_dnsmgr_lookup(peer ? peer->tohost : r->hostname, &r->us, &r->dnsmgr, sip_cfg.srvlookup ? transport : NULL); | 
 | ||||||
|  | 		/* No point in doing a DNS lookup of the register hostname if we're just going to
 | ||||||
|  | 		 * end up using an outbound proxy. obproxy_get is safe to call with either of r->call | ||||||
|  | 		 * or peer NULL. Since we're only concerned with its existence, we're not going to | ||||||
|  | 		 * bother getting a ref to the proxy*/ | ||||||
|  | 		if (!obproxy_get(r->call, peer)) { | ||||||
|  | 			ast_dnsmgr_lookup(peer ? peer->tohost : r->hostname, &r->us, &r->dnsmgr, sip_cfg.srvlookup ? transport : NULL); | ||||||
|  | 		} | ||||||
| 		if (peer) { | 		if (peer) { | ||||||
| 			peer = unref_peer(peer, "removing peer ref for dnsmgr_lookup"); | 			peer = unref_peer(peer, "removing peer ref for dnsmgr_lookup"); | ||||||
| 		} | 		} | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user