From d4ce12af69b60b527be4e6d576347bb52ba1c5e2 Mon Sep 17 00:00:00 2001 From: XieHan Date: Sat, 30 Jan 2021 16:25:39 +0800 Subject: [PATCH] remove another unnecessarycheck --- src/factory/WFDNSResolver.cc | 41 ++++++++++++++++-------------------- 1 file changed, 18 insertions(+), 23 deletions(-) diff --git a/src/factory/WFDNSResolver.cc b/src/factory/WFDNSResolver.cc index 35c27607..1bad45dd 100644 --- a/src/factory/WFDNSResolver.cc +++ b/src/factory/WFDNSResolver.cc @@ -117,32 +117,27 @@ void WFResolverTask::dispatch() if (addr_handle) { - if (addr_handle->value.addrinfo) + auto *route_manager = WFGlobal::get_route_manager(); + struct addrinfo *addrinfo = addr_handle->value.addrinfo; + struct addrinfo first; + + if (first_addr_only_ && addrinfo->ai_next) { - auto *route_manager = WFGlobal::get_route_manager(); - struct addrinfo *addrinfo = addr_handle->value.addrinfo; - struct addrinfo first; - - if (first_addr_only_ && addrinfo->ai_next) - { - first = *addrinfo; - first.ai_next = NULL; - addrinfo = &first; - } - - if (route_manager->get(type_, addrinfo, - info_, &endpoint_params_, - this->result) < 0) - { - this->state = WFT_STATE_SYS_ERROR; - this->error = errno; - } - else - this->state = WFT_STATE_SUCCESS; - - insert_dns_ = false; + first = *addrinfo; + first.ai_next = NULL; + addrinfo = &first; } + if (route_manager->get(type_, addrinfo, info_, &endpoint_params_, + this->result) < 0) + { + this->state = WFT_STATE_SYS_ERROR; + this->error = errno; + } + else + this->state = WFT_STATE_SUCCESS; + + insert_dns_ = false; dns_cache->release(addr_handle); } }