diff options
| author | Denis Vlasenko | 2007-08-09 08:04:05 +0000 |
|---|---|---|
| committer | Denis Vlasenko | 2007-08-09 08:04:05 +0000 |
| commit | 22a80d1e7d3f83b7abc0e83e71b0056c2d18ebe3 (patch) | |
| tree | 90477b681557713e700a409de6b01f4cc21ea9f2 /networking/udhcp/leases.c | |
| parent | e5dbba22afb0600404d18541613f377fde8de9f6 (diff) | |
| download | busybox-22a80d1e7d3f83b7abc0e83e71b0056c2d18ebe3.zip busybox-22a80d1e7d3f83b7abc0e83e71b0056c2d18ebe3.tar.gz | |
udhcpd: fix bug: s/memcmp() != 0/memcmp() == 0/
(by David Updegraff <dave@cray.com>)
Diffstat (limited to 'networking/udhcp/leases.c')
| -rw-r--r-- | networking/udhcp/leases.c | 21 |
1 files changed, 10 insertions, 11 deletions
diff --git a/networking/udhcp/leases.c b/networking/udhcp/leases.c index c0864ed..60e9edc 100644 --- a/networking/udhcp/leases.c +++ b/networking/udhcp/leases.c @@ -12,10 +12,10 @@ static struct dhcpOfferedAddr *oldest_expired_lease(void) { struct dhcpOfferedAddr *oldest = NULL; +// TODO: use monotonic_sec() unsigned long oldest_lease = time(0); unsigned i; - for (i = 0; i < server_config.max_leases; i++) if (oldest_lease > leases[i].expires) { oldest_lease = leases[i].expires; @@ -30,13 +30,14 @@ static void clear_lease(const uint8_t *chaddr, uint32_t yiaddr) { unsigned i, j; - for (j = 0; j < 16 && !chaddr[j]; j++); + for (j = 0; j < 16 && !chaddr[j]; j++) + continue; for (i = 0; i < server_config.max_leases; i++) - if ((j != 16 && memcmp(leases[i].chaddr, chaddr, 16) != 0) + if ((j != 16 && memcmp(leases[i].chaddr, chaddr, 16) == 0) || (yiaddr && leases[i].yiaddr == yiaddr) ) { - memset(&(leases[i]), 0, sizeof(struct dhcpOfferedAddr)); + memset(&(leases[i]), 0, sizeof(leases[i])); } } @@ -122,20 +123,18 @@ uint32_t find_address(int check_expired) struct dhcpOfferedAddr *lease = NULL; addr = server_config.start_ip; /* addr is in host order here */ - for (;addr <= server_config.end_ip; addr++) { - + for (; addr <= server_config.end_ip; addr++) { /* ie, 192.168.55.0 */ - if (!(addr & 0xFF)) continue; - + if (!(addr & 0xFF)) + continue; /* ie, 192.168.55.255 */ - if ((addr & 0xFF) == 0xFF) continue; - + if ((addr & 0xFF) == 0xFF) + continue; /* Only do if it isn't assigned as a static lease */ ret = htonl(addr); if (!reservedIp(server_config.static_leases, ret)) { /* lease is not taken */ lease = find_lease_by_yiaddr(ret); - /* no lease or it expired and we are checking for expired leases */ if ((!lease || (check_expired && lease_expired(lease))) && nobody_responds_to_arp(ret) /* it isn't used on the network */ |