[smsc95xx] Reserve headroom in received packets
authorMichael Brown <mcb30@ipxe.org>
Tue, 19 Jan 2016 00:04:33 +0000 (00:04 +0000)
committerMichael Brown <mcb30@ipxe.org>
Tue, 19 Jan 2016 00:23:06 +0000 (00:23 +0000)
commit207edc46158a91128c28de43c47cb4c1ac98e207
tree4b6c10883cdf9724b615f27794b0a2858d6de7cd
parent71b83a6d00caedb62cc62a5810f99a7a1478f2ae
[smsc95xx] Reserve headroom in received packets

Some protocols (such as ARP) may modify the received packet and re-use
the same I/O buffer for transmission of a reply.  The SMSC95XX
transmit header is larger than the receive header: the re-used I/O
buffer therefore does not have sufficient headroom for the transmit
header, and the ARP reply will therefore fail to be transmitted.  This
is essentially the same problem as in commit 2e72d10 ("[ncm] Reserve
headroom in received packets").

Fix by reserving sufficient space at the start of each received packet
to allow for the difference between the lengths of the transmit and
receive headers.

This problem is not caught by the current driver development test
suite (documented at http://ipxe.org/dev/driver), since even the large
file transfer tests tend to completely sufficiently quickly that there
is no need for the server to ever send an ARP request.  The failure
shows up only when using a very slow protocol such as RFC7440-enhanced
TFTP (as used by Windows Deployment Services).

Signed-off-by: Michael Brown <mcb30@ipxe.org>
src/drivers/net/smsc95xx.c