diff options
| author | Michael Walle <michael@walle.cc> | 2012-06-22 11:24:28 +0000 |
|---|---|---|
| committer | Joe Hershberger <joe.hershberger@ni.com> | 2012-09-24 13:17:24 -0500 |
| commit | 46c07bcf12a7d6478b5e2f3e4b4c961d84428c6c (patch) | |
| tree | 297f3479f5de9542d895485420c1818747a86928 /net | |
| parent | 7f51898c1b170e0036e6931a17ada46e2b4fd52e (diff) | |
api: net: fix length check in eth_receive()
If the requested length is too small to hold the received packet,
eth_receive() will return -1 and will leave the packet in the receive
buffers. Instead of returning an error in this case, we return the first
portion of the received packet and remove it from the receive buffers.
This fixes FreeBSD's ubldr. Without this patch it will just stop receiving
packets if the NIC receives more than PKTBUFSRX too large packets.
Signed-off-by: Michael Walle <michael@walle.cc>
Cc: Joe Hershberger <joe.hershberger@gmail.com>
Cc: Rafal Jaworowski <raj@semihalf.com>
Cc: Piotr Kruszynski <ppk@semihalf.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/eth.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/net/eth.c b/net/eth.c index 1a11ce10277..bb4d95be8ec 100644 --- a/net/eth.c +++ b/net/eth.c @@ -500,10 +500,7 @@ int eth_receive(void *packet, int length) return -1; } - if (length < eth_rcv_bufs[eth_rcv_current].length) - return -1; - - length = eth_rcv_bufs[eth_rcv_current].length; + length = min(eth_rcv_bufs[eth_rcv_current].length, length); for (i = 0; i < length; i++) p[i] = eth_rcv_bufs[eth_rcv_current].data[i]; |