udhcpc[6]: remove overzealous timeout clamping code

function                                             old     new   delta
udhcpc_main                                         2840    2826     -14
udhcpc6_main                                        2699    2670     -29

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

1 files changed, 4 insertions, 2 deletions

diff --git a/networking/udhcp/dhcpc.c b/networking/udhcp/dhcpc.c
index 55f21c187..385fc4998 100644
--- a/networking/udhcp/dhcpc.c
+++ b/networking/udhcp/dhcpc.c
@@ -1732,8 +1732,10 @@ int udhcpc_main(int argc UNUSED_PARAM, char **argv)
                                         /* paranoia: must not be too small and not prone to overflows */
                                         if (lease_seconds < 0x10)
                                                 lease_seconds = 0x10;
-                                        if (lease_seconds > 0x7fffffff / 1000)
-                                                lease_seconds = 0x7fffffff / 1000;
+                                        //if (lease_seconds > 0x7fffffff)
+                                        //      lease_seconds = 0x7fffffff;
+                                        //^^^not necessary since "timeout = lease_seconds / 2"
+                                        //does not overflow even for 0xffffffff.
                                 }
 #if ENABLE_FEATURE_UDHCPC_ARPING
                                 if (opt & OPT_a) {