diff options
| author | Andy Fleming <afleming@freescale.com> | 2010-03-29 19:42:23 +0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-03-31 10:03:16 +0400 |
| commit | 7c0d10d35f7f47d00cc5f2b85ee5e95c2b1fdb7e (patch) | |
| tree | f05d4843bd0baa72893e099107d5296168dcfac7 /drivers | |
| parent | baff42ab1494528907bf4d5870359e31711746ae (diff) | |
| download | linux-7c0d10d35f7f47d00cc5f2b85ee5e95c2b1fdb7e.tar.xz | |
gianfar: Fix a memory leak in gianfar close code
gianfar needed to ensure existence of the *skbuff arrays before
freeing the skbs in them, rather than ensuring their nonexistence.
Signed-off-by: Andy Fleming <afleming@freescale.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/gianfar.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/gianfar.c b/drivers/net/gianfar.c index 669de028d44f..c98fead8412b 100644 --- a/drivers/net/gianfar.c +++ b/drivers/net/gianfar.c @@ -1638,13 +1638,13 @@ static void free_skb_resources(struct gfar_private *priv) /* Go through all the buffer descriptors and free their data buffers */ for (i = 0; i < priv->num_tx_queues; i++) { tx_queue = priv->tx_queue[i]; - if(!tx_queue->tx_skbuff) + if(tx_queue->tx_skbuff) free_skb_tx_queue(tx_queue); } for (i = 0; i < priv->num_rx_queues; i++) { rx_queue = priv->rx_queue[i]; - if(!rx_queue->rx_skbuff) + if(rx_queue->rx_skbuff) free_skb_rx_queue(rx_queue); } |