summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid S. Miller <davem@davemloft.net>2016-01-15 22:07:13 +0100
committerDavid S. Miller <davem@davemloft.net>2016-01-15 22:07:13 +0100
commit750afbf8ee9c6a1c74a1fe5fc9852146b1d72687 (patch)
treef19b59c1fae1efa6f54d8ff8854c05e8a056ea76
parentbridge: fix lockdep addr_list_lock false positive splat (diff)
downloadlinux-750afbf8ee9c6a1c74a1fe5fc9852146b1d72687.tar.xz
linux-750afbf8ee9c6a1c74a1fe5fc9852146b1d72687.zip
bgmac: Fix reversed test of build_skb() return value.
Fixes: f1640c3ddeec ("bgmac: fix a missing check for build_skb") Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--drivers/net/ethernet/broadcom/bgmac.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/broadcom/bgmac.c b/drivers/net/ethernet/broadcom/bgmac.c
index 397415217125..06f6cffdfaf5 100644
--- a/drivers/net/ethernet/broadcom/bgmac.c
+++ b/drivers/net/ethernet/broadcom/bgmac.c
@@ -466,7 +466,7 @@ static int bgmac_dma_rx_read(struct bgmac *bgmac, struct bgmac_dma_ring *ring,
len -= ETH_FCS_LEN;
skb = build_skb(buf, BGMAC_RX_ALLOC_SIZE);
- if (unlikely(skb)) {
+ if (unlikely(!skb)) {
bgmac_err(bgmac, "build_skb failed\n");
put_page(virt_to_head_page(buf));
break;