1 files changed, 24 insertions, 10 deletions

diff --git a/networking/udhcp/packet.c b/networking/udhcp/packet.c
index da3807773..272e79df1 100644
--- a/networking/udhcp/packet.c
+++ b/networking/udhcp/packet.c
@@ -41,16 +41,17 @@ void udhcp_init_header(struct dhcpMessage *packet, char type)
 /* read a packet from socket fd, return -1 on read error, -2 on packet error */
 int udhcp_get_packet(struct dhcpMessage *packet, int fd)
 {
+#if 0
         static const char broken_vendors[][8] = {
                 "MSFT 98",
                 ""
         };
+#endif
         int bytes;
-        int i;
-        char unsigned *vendor;
+        unsigned char *vendor;
 
-        memset(packet, 0, sizeof(struct dhcpMessage));
-        bytes = read(fd, packet, sizeof(struct dhcpMessage));
+        memset(packet, 0, sizeof(*packet));
+        bytes = read(fd, packet, sizeof(*packet));
         if (bytes < 0) {
                 DEBUG("cannot read on listening socket, ignoring");
                 return -1;
@@ -62,15 +63,28 @@ int udhcp_get_packet(struct dhcpMessage *packet, int fd)
         }
         DEBUG("Received a packet");
 
-        if (packet->op == BOOTREQUEST && (vendor = get_option(packet, DHCP_VENDOR))) {
-                for (i = 0; broken_vendors[i][0]; i++) {
-                        if (vendor[OPT_LEN - 2] == (uint8_t)strlen(broken_vendors[i])
-                         && !strncmp((char*)vendor, broken_vendors[i], vendor[OPT_LEN - 2])
+        if (packet->op == BOOTREQUEST) {
+                vendor = get_option(packet, DHCP_VENDOR);
+                if (vendor) {
+#if 0
+                        int i;
+                        for (i = 0; broken_vendors[i][0]; i++) {
+                                if (vendor[OPT_LEN - 2] == (uint8_t)strlen(broken_vendors[i])
+                                 && !strncmp((char*)vendor, broken_vendors[i], vendor[OPT_LEN - 2])
+                                ) {
+                                        DEBUG("broken client (%s), forcing broadcast",
+                                                broken_vendors[i]);
+                                        packet->flags |= htons(BROADCAST_FLAG);
+                                }
+                        }
+#else
+                        if (vendor[OPT_LEN - 2] == (uint8_t)(sizeof("MSFT 98")-1)
+                         && memcmp(vendor, "MSFT 98", sizeof("MSFT 98")-1) == 0
                         ) {
-                                DEBUG("broken client (%s), forcing broadcast",
-                                        broken_vendors[i]);
+                                DEBUG("broken client (%s), forcing broadcast", "MSFT 98");
                                 packet->flags |= htons(BROADCAST_FLAG);
                         }
+#endif
                 }
         }