mv643xx_eth: fix NULL pointer dereference in rxq_process()
authorLennert Buytenhek <buytenh@wantstofly.org>
Sun, 24 Aug 2008 00:33:47 +0000 (02:33 +0200)
committerLennert Buytenhek <buytenh@marvell.com>
Sun, 24 Aug 2008 01:33:40 +0000 (03:33 +0200)
commit9e1f37724265725ad4c14fc2ef60a162dc13ac64
tree08fe8a280e33dd1c993947d3ea93ddcb8062e3b2
parent8e0b1bf6ac6c4c6dd985e586cd765aede4678bba
mv643xx_eth: fix NULL pointer dereference in rxq_process()

When we are low on memory, the assumption that every descriptor in the
receive ring will have an skbuff associated with it does not hold.

rxq_process() was assuming that if the receive descriptor it is working
on is not owned by the hardware, it can safely be processed and handed
to the networking stack.  But a descriptor in the receive ring not being
owned by the hardware can also happen when we are low on memory and did
not manage to refill the receive ring fully.

This patch changes rxq_process()'s bailout condition from "the first
receive descriptor to be processed is owned by the hardware" to "the
first receive descriptor to be processed is owned by the hardware OR
the number of valid receive descriptors in the ring is zero".

Signed-off-by: Lennert Buytenhek <buytenh@marvell.com>
drivers/net/mv643xx_eth.c