summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenys Vlasenko2018-01-16 21:47:10 +0100
committerDenys Vlasenko2018-01-16 21:47:10 +0100
commit14f13205efda0d1dc09d97ffab036162c13b94d1 (patch)
tree6e4c1929b8ca94e8fe46cb9172bbdbd5a187b038
parentef5207f093216fe6cf6f1bc7a098bcab5f435a98 (diff)
downloadbusybox-14f13205efda0d1dc09d97ffab036162c13b94d1.zip
busybox-14f13205efda0d1dc09d97ffab036162c13b94d1.tar.gz
udhcpc6: code shrink
function old new delta udhcpc6_main 2723 2727 +4 .rodata 128695 128561 -134 Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r--networking/udhcp/d6_dhcpc.c44
1 files changed, 25 insertions, 19 deletions
diff --git a/networking/udhcp/d6_dhcpc.c b/networking/udhcp/d6_dhcpc.c
index cfb2dcc..3e200eb 100644
--- a/networking/udhcp/d6_dhcpc.c
+++ b/networking/udhcp/d6_dhcpc.c
@@ -1501,10 +1501,12 @@ int udhcpc6_main(int argc UNUSED_PARAM, char **argv)
case REBINDING:
if (packet.d6_msg_type == D6_MSG_REPLY) {
uint32_t lease_seconds;
- struct d6_option *option, *iaaddr;
- int address_timeout = 0;
- int prefix_timeout = 0;
+ struct d6_option *option;
+ int address_timeout;
+ int prefix_timeout;
type_is_ok:
+ address_timeout = 0;
+ prefix_timeout = 0;
option = d6_find_option(packet.d6_options, packet_end, D6_OPT_STATUS_CODE);
if (option && (option->data[0] | option->data[1]) != 0) {
/* return to init state */
@@ -1628,6 +1630,8 @@ int udhcpc6_main(int argc UNUSED_PARAM, char **argv)
* +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
*/
if (option_mask32 & OPT_r) {
+ struct d6_option *iaaddr;
+
free(client6_data.ia_na);
client6_data.ia_na = d6_copy_option(packet.d6_options, packet_end, D6_OPT_IA_NA);
if (!client6_data.ia_na) {
@@ -1635,7 +1639,8 @@ int udhcpc6_main(int argc UNUSED_PARAM, char **argv)
continue;
}
if (client6_data.ia_na->len < (4 + 4 + 4) + (2 + 2 + 16 + 4 + 4)) {
- bb_error_msg("IA_NA option is too short:%d bytes", client6_data.ia_na->len);
+ bb_error_msg("%s option is too short:%d bytes",
+ "IA_NA", client6_data.ia_na->len);
continue;
}
iaaddr = d6_find_option(client6_data.ia_na->data + 4 + 4 + 4,
@@ -1647,7 +1652,8 @@ int udhcpc6_main(int argc UNUSED_PARAM, char **argv)
continue;
}
if (iaaddr->len < (16 + 4 + 4)) {
- bb_error_msg("IAADDR option is too short:%d bytes", iaaddr->len);
+ bb_error_msg("%s option is too short:%d bytes",
+ "IAADDR", iaaddr->len);
continue;
}
/* Note: the address is sufficiently aligned for cast:
@@ -1656,16 +1662,13 @@ int udhcpc6_main(int argc UNUSED_PARAM, char **argv)
requested_ipv6 = (struct in6_addr*) iaaddr->data;
move_from_unaligned32(lease_seconds, iaaddr->data + 16 + 4);
lease_seconds = ntohl(lease_seconds);
- /* paranoia: must not be too small and not prone to overflows */
- if (lease_seconds < 0x10)
- lease_seconds = 0x10;
/// TODO: check for 0 lease time?
+ /* paranoia: must not be prone to overflows */
if (lease_seconds > 0x7fffffff / 1000)
lease_seconds = 0x7fffffff / 1000;
- /* enter bound state */
address_timeout = lease_seconds / 2;
- bb_error_msg("lease obtained, lease time %u",
- /*inet_ntoa(temp_addr),*/ (unsigned)lease_seconds);
+ bb_error_msg("%s obtained, lease time %u",
+ "IPv6", /*inet_ntoa(temp_addr),*/ (unsigned)lease_seconds);
}
if (option_mask32 & OPT_d) {
struct d6_option *iaprefix;
@@ -1677,7 +1680,8 @@ int udhcpc6_main(int argc UNUSED_PARAM, char **argv)
continue;
}
if (client6_data.ia_pd->len < (4 + 4 + 4) + (2 + 2 + 4 + 4 + 1 + 16)) {
- bb_error_msg("IA_PD option is too short:%d bytes", client6_data.ia_pd->len);
+ bb_error_msg("%s option is too short:%d bytes",
+ "IA_PD", client6_data.ia_pd->len);
continue;
}
iaprefix = d6_find_option(client6_data.ia_pd->data + 4 + 4 + 4,
@@ -1689,22 +1693,24 @@ int udhcpc6_main(int argc UNUSED_PARAM, char **argv)
continue;
}
if (iaprefix->len < (4 + 4 + 1 + 16)) {
- bb_error_msg("IAPREFIX option is too short:%d bytes", iaprefix->len);
+ bb_error_msg("%s option is too short:%d bytes",
+ "IAPREFIX", iaprefix->len);
continue;
}
move_from_unaligned32(lease_seconds, iaprefix->data + 4);
lease_seconds = ntohl(lease_seconds);
- /* paranoia: must not be too small and not prone to overflows */
- if (lease_seconds < 0x10)
- lease_seconds = 0x10;
+ /* paranoia: must not be prone to overflows */
if (lease_seconds > 0x7fffffff / 1000)
lease_seconds = 0x7fffffff / 1000;
- /* enter bound state */
prefix_timeout = lease_seconds / 2;
- bb_error_msg("prefix obtained, lease time %u",
- /*inet_ntoa(temp_addr),*/ (unsigned)lease_seconds);
+ bb_error_msg("%s obtained, lease time %u",
+ "prefix", /*inet_ntoa(temp_addr),*/ (unsigned)lease_seconds);
}
timeout = address_timeout > prefix_timeout ? prefix_timeout : address_timeout;
+ /* paranoia: must not be too small */
+ if (timeout < 0x10)
+ timeout = 0x10;
+ /* enter bound state */
d6_run_script(&packet, state == REQUESTING ? "bound" : "renew");
state = BOUND;