slirp: Fix assertion failure on rejected DHCP requests
authorDavid Gibson <david@gibson.dropbear.id.au>
Fri, 24 Feb 2012 00:23:28 +0000 (01:23 +0100)
committerJan Kiszka <jan.kiszka@siemens.com>
Mon, 27 Feb 2012 13:55:07 +0000 (14:55 +0100)
commit90d7416ab13113ccd3891435eea8715c5f503460
treec1ef264562f2606ffaaef7bef1e8252810ecbe27
parentb87ffa163185e339f9f9f1e6dbc561e0f990442d
slirp: Fix assertion failure on rejected DHCP requests

The guest network stack might DHCPREQUEST an address that the slirp built
in dhcp server can't let it have - for example if the guest has an old
leases file from another network configuration.  In this case the dhcp
server should and does reject the request and prepares to send a DHCPNAK
to the client.

However, in this case the daddr variable in bootp_reply() is set to
0.0.0.0.  Shortly afterwards, it unconditionally attempts to pre-insert the
new client address into the ARP table.  This causes an assertion failure in
arp_address_add() because of the 0.0.0.0 address.

According to RFC2131, DHCPNAK messages for clients on the same subnet
must be sent to the broadcast address (S3.2, subpoint 2).

Cc: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
slirp/bootp.c