← Commit list

e67b80f47

Denys Vlasenko
Denys Vlasenko
2021-12-12
SUCCESS Duration: 155s

Subject

udhcpc6: fix udhcp_find_option to actually find DHCP6 options

Binary

No binary (build failed)

Architecture & Compiler Flags

Architecture: x86_64-linux-gnu Compiler: gcc (Ubuntu 11.4.0-1ubuntu1~22.04.3) 11.4.0

—

Build Log

(log yok)

Commit Diff GitHub →

networking/udhcp/common.c +22 -7networking/udhcp/common.h +5 -1networking/udhcp/d6_dhcpc.c +3 -2networking/udhcp/dhcpc.c +3 -3networking/udhcp/dhcpd.c +1 -1
networking/udhcp/common.c+22 -7
@@ -404,14 +404,29 @@ void FAST_FUNC udhcp_add_simple_option(struct dhcp_packet *packet, uint8_t code,
#endif
/* Find option 'code' in opt_list */
-struct option_set* FAST_FUNC udhcp_find_option(struct option_set *opt_list, uint8_t code)
+struct option_set* FAST_FUNC udhcp_find_option(struct option_set *opt_list, uint8_t code, bool dhcpv6)
{
- while (opt_list && opt_list->data[OPT_CODE] < code)
- opt_list = opt_list->next;
+ IF_NOT_UDHCPC6(bool dhcpv6 = 0;)
+ uint8_t cur_code;
- if (opt_list && opt_list->data[OPT_CODE] == code)
- return opt_list;
- return NULL;
+ for (;;) {
+ if (!opt_list)
+ return opt_list; /* NULL */
+ if (!dhcpv6) {
+ cur_code = opt_list->data[OPT_CODE];
+ } else {
+//FIXME: add support for code > 0xff
+ if (opt_list->data[D6_OPT_CODE] != 0)
+ return NULL;
+ cur_code = opt_list->data[D6_OPT_CODE + 1];
+ }
+ if (cur_code >= code) {
+ if (cur_code == code)
+ return opt_list;
+ return NULL;
+ }
+ opt_list = opt_list->next;
+ }
}
/* Parse string to IP in network order */
@@ -499,7 +514,7 @@ static NOINLINE void attach_option(
}
#endif
- existing = udhcp_find_option(*opt_list, optflag->code);
+ existing = udhcp_find_option(*opt_list, optflag->code, dhcpv6);
if (!existing) {
/* make a new option */
uint8_t *p = udhcp_insert_new_option(opt_list, optflag->code, length, dhcpv6);
networking/udhcp/common.h+5 -1
@@ -245,7 +245,11 @@ void udhcp_add_simple_option(struct dhcp_packet *packet, uint8_t code, uint32_t
char *dname_dec(const uint8_t *cstr, int clen, const char *pre) FAST_FUNC;
uint8_t *dname_enc(/*const uint8_t *cstr, int clen,*/ const char *src, int *retlen) FAST_FUNC;
#endif
-struct option_set *udhcp_find_option(struct option_set *opt_list, uint8_t code) FAST_FUNC;
+#if !ENABLE_UDHCPC6
+#define udhcp_find_option(opt_list, code, dhcpv6) \
+ udhcp_find_option(opt_list, code)
+#endif
+struct option_set *udhcp_find_option(struct option_set *opt_list, uint8_t code, bool dhcpv6) FAST_FUNC;
// RFC 2131 Table 5: Fields and options used by DHCP clients
//
networking/udhcp/d6_dhcpc.c+3 -2
@@ -888,7 +888,8 @@ int send_d6_release(struct in6_addr *server_ipv6, struct in6_addr *our_cur_ipv6)
if (client6_data.ia_pd)
opt_ptr = mempcpy(opt_ptr, client6_data.ia_pd, client6_data.ia_pd->len + 2+2);
/* Client-id */
- ci = udhcp_find_option(client_data.options, D6_OPT_CLIENTID);
+///vda
+ ci = udhcp_find_option(client_data.options, D6_OPT_CLIENTID, /*dhcpv6:*/ 1);
if (ci)
opt_ptr = mempcpy(opt_ptr, ci->data, D6_OPT_DATA + 2+2 + 6);
@@ -1272,7 +1273,7 @@ int udhcpc6_main(int argc UNUSED_PARAM, char **argv)
}
clientid_mac_ptr = NULL;
- if (!udhcp_find_option(client_data.options, D6_OPT_CLIENTID)) {
+ if (!udhcp_find_option(client_data.options, D6_OPT_CLIENTID, /*dhcpv6:*/ 1)) {
/* not set, set the default client ID */
clientid_mac_ptr = udhcp_insert_new_option(
&client_data.options, D6_OPT_CLIENTID,
networking/udhcp/dhcpc.c+3 -3
@@ -658,7 +658,7 @@ static void add_client_options(struct dhcp_packet *packet)
// This will be needed if we remove -V VENDOR_STR in favor of
// -x vendor:VENDOR_STR
- //if (!udhcp_find_option(packet.options, DHCP_VENDOR))
+ //if (!udhcp_find_option(packet.options, DHCP_VENDOR, /*dhcpv6:*/ 0))
// /* not set, set the default vendor ID */
// ...add (DHCP_VENDOR, "udhcp "BB_VER) opt...
}
@@ -676,7 +676,7 @@ static void add_serverid_and_clientid_options(struct dhcp_packet *packet, uint32
* If the client used a 'client identifier' when it obtained the lease,
* it MUST use the same 'client identifier' in the DHCPRELEASE message.
*/
- ci = udhcp_find_option(client_data.options, DHCP_CLIENT_ID);
+ ci = udhcp_find_option(client_data.options, DHCP_CLIENT_ID, /*dhcpv6:*/ 0);
if (ci)
udhcp_add_binary_option(packet, ci->data);
}
@@ -1328,7 +1328,7 @@ int udhcpc_main(int argc UNUSED_PARAM, char **argv)
}
clientid_mac_ptr = NULL;
- if (!(opt & OPT_C) && !udhcp_find_option(client_data.options, DHCP_CLIENT_ID)) {
+ if (!(opt & OPT_C) && !udhcp_find_option(client_data.options, DHCP_CLIENT_ID, /*dhcpv6:*/ 0)) {
/* not suppressed and not set, create default client ID */
clientid_mac_ptr = udhcp_insert_new_option(
&client_data.options, DHCP_CLIENT_ID,
networking/udhcp/dhcpd.c+1 -1
@@ -935,7 +935,7 @@ int udhcpd_main(int argc UNUSED_PARAM, char **argv)
bb_simple_info_msg("started, v"BB_VER);
- option = udhcp_find_option(server_data.options, DHCP_LEASE_TIME);
+ option = udhcp_find_option(server_data.options, DHCP_LEASE_TIME, /*dhcpv6:*/ 0);
server_data.max_lease_sec = DEFAULT_LEASE_TIME;
if (option) {
move_from_unaligned32(server_data.max_lease_sec, option->data + OPT_DATA);