From 6884f665bd7bc101f56ff9047afaffbc06dc99e2 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Fri, 23 Nov 2007 00:08:54 +0000 Subject: dhcp: stop using magic constants; use (htonl(CONST) != a) - it's smaller function old new delta udhcp_get_packet 146 134 -12 get_raw_packet 368 353 -15 --- networking/udhcp/packet.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) (limited to 'networking/udhcp/packet.c') diff --git a/networking/udhcp/packet.c b/networking/udhcp/packet.c index 41cd32135..0abe851a4 100644 --- a/networking/udhcp/packet.c +++ b/networking/udhcp/packet.c @@ -57,7 +57,7 @@ int udhcp_get_packet(struct dhcpMessage *packet, int fd) return -1; } - if (ntohl(packet->cookie) != DHCP_MAGIC) { + if (packet->cookie != htonl(DHCP_MAGIC)) { bb_error_msg("received bogus message, ignoring"); return -2; } @@ -123,7 +123,6 @@ uint16_t udhcp_checksum(void *addr, int count) /* Construct a ip/udp header for a packet, and specify the source and dest hardware address */ -void BUG_sizeof_struct_udp_dhcp_packet_must_be_576(void); int udhcp_raw_packet(struct dhcpMessage *payload, uint32_t source_ip, int source_port, uint32_t dest_ip, int dest_port, const uint8_t *dest_arp, int ifindex) @@ -169,9 +168,6 @@ int udhcp_raw_packet(struct dhcpMessage *payload, packet.ip.ttl = IPDEFTTL; packet.ip.check = udhcp_checksum(&(packet.ip), sizeof(packet.ip)); - if (sizeof(struct udp_dhcp_packet) != 576) - BUG_sizeof_struct_udp_dhcp_packet_must_be_576(); - result = sendto(fd, &packet, sizeof(struct udp_dhcp_packet), 0, (struct sockaddr *) &dest, sizeof(dest)); if (result <= 0) { -- cgit v1.2.3