emo/ndhc
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Validate that server reply has the client hardware address field set

Browse Source 
to be equal to our MAC address.  Win7 verifies this field, so it should
be safe enough to check it, too.
This commit is contained in:
Nicholas J. Kain 2011-09-01 23:05:56 -04:00
parent ab4a2e0b97
commit c4b4938a59
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
ndhc/dhcp.c
View File

@ -503,6 +503,15 @@ static int validate_dhcp_packet(struct client_state_t *cs, int len,
packet->xid, cs->xid); packet->xid, cs->xid);
return 0; return 0;
} }
if (memcmp(packet->chaddr, client_config.arp, sizeof client_config.arp)) {
log_line("Packet client MAC %.2hhx:%.2hhx:%.2hhx:%.2hhx:%.2hhx:%.2hhx does not equal our MAC %.2hhx:%.2hhx:%.2hhx:%.2hhx:%.2hhx:%.2hhx. Ignoring it.",
packet->chaddr[0], packet->chaddr[1], packet->chaddr[2],
packet->chaddr[3], packet->chaddr[4], packet->chaddr[5],
client_config.arp[0], client_config.arp[1],
client_config.arp[2], client_config.arp[3],
client_config.arp[4], client_config.arp[5]);
return 0;
}
*msgtype = get_option_msgtype(packet); *msgtype = get_option_msgtype(packet);
if (!*msgtype) { if (!*msgtype) {
log_line("Packet does not specify a DHCP message type. Ignoring."); log_line("Packet does not specify a DHCP message type. Ignoring.");