[Asterisk-code-review] Fix res search usage (asterisk[master])

Joshua Colp asteriskteam at digium.com
Wed Jun 8 14:43:35 CDT 2016


Joshua Colp has submitted this change and it was merged.

Change subject: Fix res_search usage
......................................................................


Fix res_search usage

Resolver state is not part of res_search API. This fixes
compilation error:

dns.c:261:8: error: too many arguments to function 'res_search'
  ret = res_search(&dns_state,

Change-Id: Ia600a58557040df83f744da3dde23225293845a5
---
M main/dns.c
1 file changed, 1 insertion(+), 3 deletions(-)

Approvals:
  Mark Michelson: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved; Verified



diff --git a/main/dns.c b/main/dns.c
index 9622794..fa94089 100644
--- a/main/dns.c
+++ b/main/dns.c
@@ -254,12 +254,10 @@
 {
 
 	int ret = AST_DNS_SEARCH_FAILURE;
-	struct __res_state dns_state;
 
 	ast_mutex_lock(&res_lock);
 	res_init();
-	ret = res_search(&dns_state,
-	                 dname,
+	ret = res_search(dname,
 	                 rr_class,
 	                 rr_type,
 	                 dns_response,

-- 
To view, visit https://gerrit.asterisk.org/2935
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia600a58557040df83f744da3dde23225293845a5
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Timo Teräs <timo.teras at iki.fi>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>



More information about the asterisk-code-review mailing list