move the following auto *addr above the if and check whether the addr…

… is nullptr
vesoft-inc · Apr 19, 2024 · a641d3f · a641d3f
1 parent 73ff1a2
commit a641d3f
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/common/network/NetworkUtils.cpp b/src/common/network/NetworkUtils.cpp
@@ -65,8 +65,9 @@ StatusOr<std::vector<std::pair<std::string, std::string>>> NetworkUtils::listDev
     return Status::Error("%s", ::strerror(errno));
   }
   for (auto* ifa = iflist; ifa != nullptr; ifa = ifa->ifa_next) {
+    auto* addr = ifa->ifa_addr;
     // Skip non-IPv4 devices
-    if (nullptr == ifa->ifa_addr || ifa->ifa_addr->sa_family != AF_INET) {
+    if (nullptr == addr || addr->sa_family != AF_INET) {
       continue;
     }
     auto* addr = reinterpret_cast<struct sockaddr_in*>(ifa->ifa_addr);