]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
network: address: drop unnecessary call of address_get()
authorYu Watanabe <watanabe.yu+github@gmail.com>
Thu, 24 Feb 2022 02:24:08 +0000 (11:24 +0900)
committerYu Watanabe <watanabe.yu+github@gmail.com>
Fri, 25 Feb 2022 02:01:50 +0000 (11:01 +0900)
As here `req->address` and the result `existing` are always equivalent.

src/network/networkd-address.c

index bcb23c90e1e6bec5bf0cb109141bee1d1bae3940..4369d09fa0909b2b1a360852619fe3fb6747e354 100644 (file)
@@ -1270,30 +1270,24 @@ static bool address_is_ready_to_configure(Link *link, const Address *address) {
 }
 
 int request_process_address(Request *req) {
-        Address *existing;
+        Address *address;
         Link *link;
         int r;
 
         assert(req);
-        assert(req->link);
-        assert(req->address);
         assert(req->type == REQUEST_TYPE_ADDRESS);
 
-        link = req->link;
+        address = ASSERT_PTR(req->address);
+        link = ASSERT_PTR(req->link);
 
-        r = address_get(link, req->address, &existing);
-        if (r < 0)
-                return log_link_warning_errno(link, r, "Failed to get address: %m");
-
-        if (!address_is_ready_to_configure(link, existing))
+        if (!address_is_ready_to_configure(link, address))
                 return 0;
 
-        r = address_configure(req->address, link, req->netlink_handler);
+        r = address_configure(address, link, req->netlink_handler);
         if (r < 0)
                 return log_link_warning_errno(link, r, "Failed to configure address: %m");
 
-        address_enter_configuring(existing);
-
+        address_enter_configuring(address);
         return 1;
 }