diff options
| author | Sreenivasa Honnur <Sreenivasa.Honnur@neterion.com> | 2008-04-25 13:21:40 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-05-01 14:44:34 -0700 |
| commit | d7380f30c886bf78319f721a13fd61b3fd094849 (patch) | |
| tree | 5db55247ab63a83ef8a77c254ca901f08dcaa877 /drivers | |
| parent | dcbfddf29347e68493d959d9592e5fea654fd624 (diff) | |
| download | lwn-d7380f30c886bf78319f721a13fd61b3fd094849.tar.gz lwn-d7380f30c886bf78319f721a13fd61b3fd094849.zip | |
S2io: Fix memory leak during free_tx_buffers
commit b35b3b49fc6750806964048b31799c8782980ef9 upstream
- Fix the memory leak during free_tx_buffers.
Signed-off-by: Santosh Rastapur <santosh.rastapur@neterion.com>
Signed-off-by: Ramkrishna Vepa <ram.vepa@neterion.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/s2io.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/s2io.c b/drivers/net/s2io.c index c082cf0b1ac6..51a63da05fc7 100644 --- a/drivers/net/s2io.c +++ b/drivers/net/s2io.c @@ -2339,7 +2339,7 @@ static void free_tx_buffers(struct s2io_nic *nic) for (i = 0; i < config->tx_fifo_num; i++) { unsigned long flags; spin_lock_irqsave(&mac_control->fifos[i].tx_lock, flags); - for (j = 0; j < config->tx_cfg[i].fifo_len - 1; j++) { + for (j = 0; j < config->tx_cfg[i].fifo_len; j++) { txdp = (struct TxD *) \ mac_control->fifos[i].list_info[j].list_virt_addr; skb = s2io_txdl_getskb(&mac_control->fifos[i], txdp, j); |