forcedeth: remove unnecessary checks before kfree
authorSzymon Janc <szymon@janc.net.pl>
Sat, 27 Nov 2010 08:39:44 +0000 (08:39 +0000)
committerDavid S. Miller <davem@davemloft.net>
Mon, 29 Nov 2010 02:06:57 +0000 (18:06 -0800)
Signed-off-by: Szymon Janc <szymon@janc.net.pl>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/forcedeth.c

index 87757c89caef77551dbd020aeaec779debd3f817..81722fb51a367c4c3e16e6b69124c0a15ec85dfa 100644 (file)
@@ -1013,10 +1013,8 @@ static void free_rings(struct net_device *dev)
                        pci_free_consistent(np->pci_dev, sizeof(struct ring_desc_ex) * (np->rx_ring_size + np->tx_ring_size),
                                            np->rx_ring.ex, np->ring_addr);
        }
-       if (np->rx_skb)
-               kfree(np->rx_skb);
-       if (np->tx_skb)
-               kfree(np->tx_skb);
+       kfree(np->rx_skb);
+       kfree(np->tx_skb);
 }
 
 static int using_multi_irqs(struct net_device *dev)
@@ -4442,10 +4440,9 @@ static int nv_set_ringparam(struct net_device *dev, struct ethtool_ringparam* ri
                                pci_free_consistent(np->pci_dev, sizeof(struct ring_desc_ex) * (ring->rx_pending + ring->tx_pending),
                                                    rxtx_ring, ring_addr);
                }
-               if (rx_skbuff)
-                       kfree(rx_skbuff);
-               if (tx_skbuff)
-                       kfree(tx_skbuff);
+
+               kfree(rx_skbuff);
+               kfree(tx_skbuff);
                goto exit;
        }