diff options
author | Florian Fainelli <florian@openwrt.org> | 2014-01-15 16:04:26 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-01-16 19:22:54 -0500 |
commit | 31cf344caf444ca7411d89c8ac907d886eeab1a7 (patch) | |
tree | c8cad9f43e4df16ff955b2d9b1da60f72965f219 | |
parent | 4f8d9f3ce0e52adf2cb4e0661f06ef8cfdc97cfe (diff) |
r6040: use ETH_ZLEN instead of MISR for SKB length checking
Ever since this driver was merged the following code was included:
if (skb->len < MISR)
skb->len = MISR;
MISR is defined to 0x3C which is also equivalent to ETH_ZLEN, but use
ETH_ZLEN directly which is exactly what we want to be checking for.
Reported-by: Marc Volovic <marcv@ezchip.com>
Signed-off-by: Florian Fainelli <florian@openwrt.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/rdc/r6040.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ethernet/rdc/r6040.c b/drivers/net/ethernet/rdc/r6040.c index c6cd1d05f8ea..819b74cefd64 100644 --- a/drivers/net/ethernet/rdc/r6040.c +++ b/drivers/net/ethernet/rdc/r6040.c | |||
@@ -835,8 +835,8 @@ static netdev_tx_t r6040_start_xmit(struct sk_buff *skb, | |||
835 | /* Set TX descriptor & Transmit it */ | 835 | /* Set TX descriptor & Transmit it */ |
836 | lp->tx_free_desc--; | 836 | lp->tx_free_desc--; |
837 | descptr = lp->tx_insert_ptr; | 837 | descptr = lp->tx_insert_ptr; |
838 | if (skb->len < MISR) | 838 | if (skb->len < ETH_ZLEN) |
839 | descptr->len = MISR; | 839 | descptr->len = ETH_ZLEN; |
840 | else | 840 | else |
841 | descptr->len = skb->len; | 841 | descptr->len = skb->len; |
842 | 842 | ||