treewide: refactor valid to valid_lt (lifetime)
authorPaul Donald <newtwen@gmail.com>
Tue, 9 Apr 2024 03:04:03 +0000 (05:04 +0200)
committerChristian Marangi <ansuelsmth@gmail.com>
Tue, 7 May 2024 22:06:07 +0000 (00:06 +0200)
Refactor "valid" (valid what?) to "valid_lt".

Signed-off-by: Paul Donald <newtwen@gmail.com>
Reviewed-by: Daniel Golle <daniel@makrotopia.org>
[ improve commit title ]
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
src/dhcpv6-ia.c
src/dhcpv6.c
src/dhcpv6.h
src/netlink.c
src/odhcpd.c
src/odhcpd.h
src/router.c

index 6a9098e264069de288e76261d8ada2b86c32e3b2..39a316ac7ab51b876bf9feb9d05a6b90d34bf4c8 100644 (file)
@@ -226,7 +226,7 @@ void dhcpv6_ia_enum_addrs(struct interface *iface, struct dhcp_assignment *c,
 
        for (size_t i = 0; i < addrlen; ++i) {
                struct in6_addr addr;
-               uint32_t preferred_lt, valid;
+               uint32_t preferred_lt, valid_lt;
                int prefix = c->managed ? addrs[i].prefix : c->length;
 
                if (!valid_addr(&addrs[i], now))
@@ -243,7 +243,7 @@ void dhcpv6_ia_enum_addrs(struct interface *iface, struct dhcp_assignment *c,
 
                addr = addrs[i].addr.in6;
                preferred_lt = addrs[i].preferred_lt;
-               valid = addrs[i].valid;
+               valid_lt = addrs[i].valid_lt;
 
                if (c->flags & OAF_DHCPV6_NA) {
                        if (!ADDR_ENTRY_VALID_IA_ADDR(iface, i, m, addrs))
@@ -268,13 +268,13 @@ void dhcpv6_ia_enum_addrs(struct interface *iface, struct dhcp_assignment *c,
                if (preferred_lt != UINT32_MAX)
                        preferred_lt -= now;
 
-               if (valid > (uint32_t)c->valid_until)
-                       valid = c->valid_until;
+               if (valid_lt > (uint32_t)c->valid_until)
+                       valid_lt = c->valid_until;
 
-               if (valid != UINT32_MAX)
-                       valid -= now;
+               if (valid_lt != UINT32_MAX)
+                       valid_lt -= now;
 
-               func(&addr, prefix, preferred_lt, valid, arg);
+               func(&addr, prefix, preferred_lt, valid_lt, arg);
        }
 }
 
@@ -289,7 +289,7 @@ struct write_ctxt {
 };
 
 static void dhcpv6_write_ia_addrhosts(struct in6_addr *addr, int prefix, _unused uint32_t pref_lt,
-                               _unused uint32_t valid, void *arg)
+                               _unused uint32_t valid_lt, void *arg)
 {
        struct write_ctxt *ctxt = (struct write_ctxt *)arg;
        char ipbuf[INET6_ADDRSTRLEN];
@@ -309,7 +309,7 @@ static void dhcpv6_write_ia_addrhosts(struct in6_addr *addr, int prefix, _unused
 }
 
 static void dhcpv6_write_ia_addr(struct in6_addr *addr, int prefix, _unused uint32_t pref_lt,
-                               _unused uint32_t valid, void *arg)
+                               _unused uint32_t valid_lt, void *arg)
 {
        struct write_ctxt *ctxt = (struct write_ctxt *)arg;
        char ipbuf[INET6_ADDRSTRLEN];
@@ -682,10 +682,10 @@ static void managed_handle_pd_data(struct ustream *s, _unused int bytes_new)
                                continue;
 
                        x = strtok_r(NULL, ",", &saveptr2);
-                       if (sscanf(x, "%u", &n->valid) < 1)
+                       if (sscanf(x, "%u", &n->valid_lt) < 1)
                                continue;
 
-                       if (n->preferred_lt > n->valid)
+                       if (n->preferred_lt > n->valid_lt)
                                continue;
 
                        if (UINT32_MAX - now < n->preferred_lt)
@@ -693,10 +693,10 @@ static void managed_handle_pd_data(struct ustream *s, _unused int bytes_new)
                        else
                                n->preferred_lt += now;
 
-                       if (UINT32_MAX - now < n->valid)
-                               n->valid = UINT32_MAX;
+                       if (UINT32_MAX - now < n->valid_lt)
+                               n->valid_lt = UINT32_MAX;
                        else
-                               n->valid += now;
+                               n->valid_lt += now;
 
                        n->dprefix = 0;
 
@@ -1047,14 +1047,14 @@ static size_t build_ia(uint8_t *buf, size_t buflen, uint16_t status,
                        preferred_lt = iface->preferred_lifetime;
                }
 
-               uint32_t valid = leasetime;
+               uint32_t valid_lt = leasetime;
 
                struct odhcpd_ipaddr *addrs = (a->managed) ? a->managed : iface->addr6;
                size_t addrlen = (a->managed) ? (size_t)a->managed_size : iface->addr6_len;
                size_t m = get_preferred_addr(addrs, addrlen);
 
                for (size_t i = 0; i < addrlen; ++i) {
-                       uint32_t prefix_preferred_lt, prefix_valid;
+                       uint32_t prefix_preferred_lt, prefix_valid_lt;
 
                        if (!valid_addr(&addrs[i], now))
                                continue;
@@ -1069,7 +1069,7 @@ static size_t build_ia(uint8_t *buf, size_t buflen, uint16_t status,
                        }
 
                        prefix_preferred_lt = addrs[i].preferred_lt;
-                       prefix_valid = addrs[i].valid;
+                       prefix_valid_lt = addrs[i].valid_lt;
 
                        if (prefix_preferred_lt != UINT32_MAX)
                                prefix_preferred_lt -= now;
@@ -1077,21 +1077,21 @@ static size_t build_ia(uint8_t *buf, size_t buflen, uint16_t status,
                        if (prefix_preferred_lt > preferred_lt)
                                prefix_preferred_lt = preferred_lt;
 
-                       if (prefix_valid != UINT32_MAX)
-                               prefix_valid -= now;
+                       if (prefix_valid_lt != UINT32_MAX)
+                               prefix_valid_lt -= now;
 
-                       if (prefix_valid > leasetime)
-                               prefix_valid = leasetime;
+                       if (prefix_valid_lt > leasetime)
+                               prefix_valid_lt = leasetime;
 
-                       if (prefix_preferred_lt > prefix_valid)
-                               prefix_preferred_lt = prefix_valid;
+                       if (prefix_preferred_lt > prefix_valid_lt)
+                               prefix_preferred_lt = prefix_valid_lt;
 
                        if (a->flags & OAF_DHCPV6_PD) {
                                struct dhcpv6_ia_prefix o_ia_p = {
                                        .type = htons(DHCPV6_OPT_IA_PREFIX),
                                        .len = htons(sizeof(o_ia_p) - 4),
                                        .preferred_lt = htonl(prefix_preferred_lt),
-                                       .valid = htonl(prefix_valid),
+                                       .valid_lt = htonl(prefix_valid_lt),
                                        .prefix = (a->managed_size) ? addrs[i].prefix : a->length,
                                        .addr = addrs[i].addr.in6,
                                };
@@ -1115,7 +1115,7 @@ static size_t build_ia(uint8_t *buf, size_t buflen, uint16_t status,
                                        .len = htons(sizeof(o_ia_a) - 4),
                                        .addr = addrs[i].addr.in6,
                                        .preferred_lt = htonl(prefix_preferred_lt),
-                                       .valid = htonl(prefix_valid)
+                                       .valid_lt = htonl(prefix_valid_lt)
                                };
 
                                o_ia_a.addr.s6_addr32[2] = htonl(a->assigned_host_id >> 32);
@@ -1136,14 +1136,14 @@ static size_t build_ia(uint8_t *buf, size_t buflen, uint16_t status,
                                if (prefix_preferred_lt < preferred_lt)
                                        preferred_lt = prefix_preferred_lt;
 
-                               if (prefix_valid < valid)
-                                       valid = prefix_valid;
+                               if (prefix_valid_lt < valid_lt)
+                                       valid_lt = prefix_valid_lt;
                        }
                }
 
                if (!INFINITE_VALID(a->valid_until))
                        /* UINT32_MAX is considered as infinite leasetime */
-                       a->valid_until = (valid == UINT32_MAX) ? 0 : valid + now;
+                       a->valid_until = (valid_lt == UINT32_MAX) ? 0 : valid_lt + now;
 
                if (!INFINITE_VALID(a->preferred_until))
                        /* UINT32_MAX is considered as infinite leasetime */
@@ -1212,7 +1212,7 @@ static size_t build_ia(uint8_t *buf, size_t buflen, uint16_t status,
                                                .type = htons(DHCPV6_OPT_IA_PREFIX),
                                                .len = htons(sizeof(o_ia_p) - 4),
                                                .preferred_lt = 0,
-                                               .valid = 0,
+                                               .valid_lt = 0,
                                                .prefix = ia_p->prefix,
                                                .addr = ia_p->addr,
                                        };
@@ -1228,7 +1228,7 @@ static size_t build_ia(uint8_t *buf, size_t buflen, uint16_t status,
                                                .len = htons(sizeof(o_ia_a) - 4),
                                                .addr = ia_a->addr,
                                                .preferred_lt = 0,
-                                               .valid = 0,
+                                               .valid_lt = 0,
                                        };
 
                                        if (buflen < ia_len + sizeof(o_ia_a))
@@ -1253,7 +1253,7 @@ struct log_ctxt {
 };
 
 static void dhcpv6_log_ia_addr(struct in6_addr *addr, int prefix, _unused uint32_t pref_lt,
-                               _unused uint32_t valid, void *arg)
+                               _unused uint32_t valid_lt, void *arg)
 {
        struct log_ctxt *ctxt = (struct log_ctxt *)arg;
        char addrbuf[INET6_ADDRSTRLEN];
index b0e5a79c21244cf43ed325629bfe37335e20e15e..58b7d9fa7a3fbcb49d4b3e17d515daa2d8b5e9d3 100644 (file)
@@ -770,7 +770,7 @@ static struct odhcpd_ipaddr *relay_link_address(struct interface *iface)
        time_t now = odhcpd_time();
 
        for (size_t i = 0; i < iface->addr6_len; i++) {
-               if (iface->addr6[i].valid <= (uint32_t)now)
+               if (iface->addr6[i].valid_lt <= (uint32_t)now)
                        continue;
 
                if (iface->addr6[i].preferred_lt > (uint32_t)now) {
@@ -778,7 +778,7 @@ static struct odhcpd_ipaddr *relay_link_address(struct interface *iface)
                        break;
                }
 
-               if (!addr || (iface->addr6[i].valid > addr->valid))
+               if (!addr || (iface->addr6[i].valid_lt > addr->valid_lt))
                        addr = &iface->addr6[i];
        }
 
index 8681d7fd2a7f0db3d7893138e876b4d084a217a0..b925928d09ae1d669da7a94b8c67fbfdb6a7dc3f 100644 (file)
@@ -131,7 +131,7 @@ struct dhcpv6_ia_prefix {
        uint16_t type;
        uint16_t len;
        uint32_t preferred_lt;
-       uint32_t valid;
+       uint32_t valid_lt;
        uint8_t prefix;
        struct in6_addr addr;
 } _packed;
@@ -141,7 +141,7 @@ struct dhcpv6_ia_addr {
        uint16_t len;
        struct in6_addr addr;
        uint32_t preferred_lt;
-       uint32_t valid;
+       uint32_t valid_lt;
 } _packed;
 
 struct dhcpv6_cer_id {
index cf77035b1739b0b26be48f169af3a7368fb6f5a4..c33e3eae55576dbea9134435a91801732ef0a2d1 100644 (file)
@@ -207,7 +207,7 @@ static void refresh_iface_addr6(int ifindex)
                                if (!IN6_ARE_ADDR_EQUAL(&addr[i].addr.in6, &iface->addr6[i].addr.in6) ||
                                    addr[i].prefix != iface->addr6[i].prefix ||
                                    (addr[i].preferred_lt > (uint32_t)now) != (iface->addr6[i].preferred_lt > (uint32_t)now) ||
-                                   addr[i].valid < iface->addr6[i].valid || addr[i].preferred_lt < iface->addr6[i].preferred_lt)
+                                   addr[i].valid_lt < iface->addr6[i].valid_lt || addr[i].preferred_lt < iface->addr6[i].preferred_lt)
                                        change = true;
                        }
 
@@ -230,7 +230,7 @@ static void refresh_iface_addr6(int ifindex)
                                for (size_t i = 0; i < iface->addr6_len; ++i) {
                                        bool removed = true;
 
-                                       if (iface->addr6[i].valid <= (uint32_t)now)
+                                       if (iface->addr6[i].valid_lt <= (uint32_t)now)
                                                continue;
 
                                        for (ssize_t j = 0; removed && j < len; ++j) {
@@ -258,7 +258,7 @@ static void refresh_iface_addr6(int ifindex)
                                                iface->invalid_addr6 = new_invalid_addr6;
                                                iface->invalid_addr6_len++;
                                                memcpy(&iface->invalid_addr6[pos], &iface->addr6[i], sizeof(*iface->invalid_addr6));
-                                               iface->invalid_addr6[pos].valid = iface->invalid_addr6[pos].preferred_lt = (uint32_t)now;
+                                               iface->invalid_addr6[pos].valid_lt = iface->invalid_addr6[pos].preferred_lt = (uint32_t)now;
 
                                                if (iface->invalid_addr6[pos].prefix < 64)
                                                        iface->invalid_addr6[pos].prefix = 64;
@@ -631,7 +631,7 @@ static int cb_addr_valid(struct nl_msg *msg, void *arg)
                struct ifa_cacheinfo *ifc = nla_data(nla[IFA_CACHEINFO]);
 
                addrs[ctxt->ret].preferred_lt = ifc->ifa_prefered;
-               addrs[ctxt->ret].valid = ifc->ifa_valid;
+               addrs[ctxt->ret].valid_lt = ifc->ifa_valid;
        }
 
        if (ifa->ifa_flags & IFA_F_DEPRECATED)
@@ -752,8 +752,8 @@ ssize_t netlink_get_interface_addrs(int ifindex, bool v6, struct odhcpd_ipaddr *
                if (addr[i].preferred_lt < UINT32_MAX - now)
                        addr[i].preferred_lt += now;
 
-               if (addr[i].valid < UINT32_MAX - now)
-                       addr[i].valid += now;
+               if (addr[i].valid_lt < UINT32_MAX - now)
+                       addr[i].valid_lt += now;
        }
 
 free:
@@ -1124,16 +1124,16 @@ int netlink_setup_addr(struct odhcpd_ipaddr *addr,
                        cinfo.ifa_prefered = preferred_lt;
                }
 
-               if (addr->valid) {
-                       int64_t valid = addr->valid - now;
-                       if (valid <= 0) {
+               if (addr->valid_lt) {
+                       int64_t valid_lt = addr->valid_lt - now;
+                       if (valid_lt <= 0) {
                                nlmsg_free(msg);
                                return -1;
                        }
-                       else if (valid > UINT32_MAX)
-                               valid = UINT32_MAX;
+                       else if (valid_lt > UINT32_MAX)
+                               valid_lt = UINT32_MAX;
 
-                       cinfo.ifa_valid = valid;
+                       cinfo.ifa_valid = valid_lt;
                }
 
                nla_put(msg, IFA_CACHEINFO, sizeof(cinfo), &cinfo);
index e2e89c0acb544943f16ca6997f94e0d5955c300a..0849b4343577bc1ba6856eb642e5f7fe23e8b8c8 100644 (file)
@@ -269,7 +269,7 @@ int odhcpd_get_interface_dns_addr(const struct interface *iface, struct in6_addr
                return -1;
 
        for (size_t i = 0; i < iface->addr6_len; ++i) {
-               if (iface->addr6[i].valid <= (uint32_t)now)
+               if (iface->addr6[i].valid_lt <= (uint32_t)now)
                        continue;
 
                if (m < 0) {
index df27c14d3d502201ae68ea0caff6d57133b8c926..7a82e9823edcebd72f0e3f2eced88ca1463b7780 100644 (file)
@@ -127,7 +127,7 @@ struct odhcpd_ipaddr {
        union if_addr addr;
        uint8_t prefix;
        uint32_t preferred_lt;
-       uint32_t valid;
+       uint32_t valid_lt;
 
        union {
                /* ipv6 only */
index 55f3acb0b2b52e7582afd83a127ca42e105df136..6a9e19d77cf382f70dcc70db56949158532c435a 100644 (file)
@@ -301,7 +301,7 @@ static bool router_icmpv6_valid(struct sockaddr_in6 *source, uint8_t *data, size
 static bool parse_routes(struct odhcpd_ipaddr *n, ssize_t len)
 {
        struct odhcpd_ipaddr p = { .addr.in6 = IN6ADDR_ANY_INIT, .prefix = 0,
-                                       .dprefix = 0, .preferred_lt = 0, .valid = 0};
+                                       .dprefix = 0, .preferred_lt = 0, .valid_lt = 0};
        bool found_default = false;
        char line[512], ifname[16];
 
@@ -550,12 +550,12 @@ static int send_router_advert(struct interface *iface, const struct in6_addr *fr
                struct odhcpd_ipaddr *addr = &addrs[i];
                struct nd_opt_prefix_info *p = NULL;
                uint32_t preferred_lt = 0;
-               uint32_t valid = 0;
+               uint32_t valid_lt = 0;
 
-               if (addr->prefix > 96 || (i < valid_addr_cnt && addr->valid <= (uint32_t)now)) {
-                       syslog(LOG_INFO, "Address %s (prefix %d, valid %u) not suitable as RA prefix on %s",
+               if (addr->prefix > 96 || (i < valid_addr_cnt && addr->valid_lt <= (uint32_t)now)) {
+                       syslog(LOG_INFO, "Address %s (prefix %d, valid-lifetime %u) not suitable as RA prefix on %s",
                                inet_ntop(AF_INET6, &addr->addr.in6, buf, sizeof(buf)), addr->prefix,
-                               addr->valid, iface->name);
+                               addr->valid_lt, iface->name);
                        continue;
                }
 
@@ -595,17 +595,17 @@ static int send_router_advert(struct interface *iface, const struct in6_addr *fr
                                preferred_lt = iface->preferred_lifetime;
                }
 
-               if (addr->valid > (uint32_t)now) {
-                       valid = TIME_LEFT(addr->valid, now);
+               if (addr->valid_lt > (uint32_t)now) {
+                       valid_lt = TIME_LEFT(addr->valid_lt, now);
 
-                       if (iface->ra_useleasetime && valid > iface->dhcp_leasetime)
-                               valid = iface->dhcp_leasetime;
+                       if (iface->ra_useleasetime && valid_lt > iface->dhcp_leasetime)
+                               valid_lt = iface->dhcp_leasetime;
                }
 
-               if (minvalid > valid)
-                       minvalid = valid;
+               if (minvalid > valid_lt)
+                       minvalid = valid_lt;
 
-               if ((!IN6_IS_ADDR_ULA(&addr->addr.in6) || iface->default_router) && valid)
+               if ((!IN6_IS_ADDR_ULA(&addr->addr.in6) || iface->default_router) && valid_lt)
                        valid_prefix = true;
 
                odhcpd_bmemcpy(&p->nd_opt_pi_prefix, &addr->addr.in6,
@@ -621,7 +621,7 @@ static int send_router_advert(struct interface *iface, const struct in6_addr *fr
                if (iface->ra_advrouter)
                        p->nd_opt_pi_flags_reserved |= ND_OPT_PI_FLAG_RADDR;
                p->nd_opt_pi_preferred_time = htonl(preferred_lt);
-               p->nd_opt_pi_valid_time = htonl(valid);
+               p->nd_opt_pi_valid_time = htonl(valid_lt);
        }
 
        iov[IOV_RA_PFXS].iov_base = (char *)pfxs;
@@ -780,12 +780,12 @@ pref64_out:
        for (ssize_t i = 0; i < valid_addr_cnt; ++i) {
                struct odhcpd_ipaddr *addr = &addrs[i];
                struct nd_opt_route_info *tmp;
-               uint32_t valid;
+               uint32_t valid_lt;
 
-               if (addr->dprefix >= 64 || addr->dprefix == 0 || addr->valid <= (uint32_t)now) {
-                       syslog(LOG_INFO, "Address %s (dprefix %d, valid %u) not suitable as RA route on %s",
+               if (addr->dprefix >= 64 || addr->dprefix == 0 || addr->valid_lt <= (uint32_t)now) {
+                       syslog(LOG_INFO, "Address %s (dprefix %d, valid-lifetime %u) not suitable as RA route on %s",
                                inet_ntop(AF_INET6, &addr->addr.in6, buf, sizeof(buf)),
-                               addr->dprefix, addr->valid, iface->name);
+                               addr->dprefix, addr->valid_lt, iface->name);
 
                        continue; /* Address not suitable */
                }
@@ -822,8 +822,8 @@ pref64_out:
                else if (iface->route_preference > 0)
                        routes[routes_cnt].flags |= ND_RA_PREF_HIGH;
 
-               valid = TIME_LEFT(addr->valid, now);
-               routes[routes_cnt].lifetime = htonl(valid < lifetime ? valid : lifetime);
+               valid_lt = TIME_LEFT(addr->valid_lt, now);
+               routes[routes_cnt].lifetime = htonl(valid_lt < lifetime ? valid_lt : lifetime);
                routes[routes_cnt].addr[0] = addr->addr.in6.s6_addr32[0];
                routes[routes_cnt].addr[1] = addr->addr.in6.s6_addr32[1];
                routes[routes_cnt].addr[2] = 0;