diff options
| author | Don Fry <brazilnut@us.ibm.com> | 2006-05-25 16:22:40 -0700 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2006-05-26 21:33:18 -0400 |
| commit | a24b163b7c16f9e30e726319115e45ed6e683582 (patch) | |
| tree | 64eedaff752008bb4a5622d1e8eac3d906ccf220 | |
| parent | 80871e63e410c53524ef47eaf475c13e0f164ea6 (diff) | |
| download | op-kernel-dev-a24b163b7c16f9e30e726319115e45ed6e683582.zip op-kernel-dev-a24b163b7c16f9e30e726319115e45ed6e683582.tar.gz | |
[PATCH] pcnet32: remove incorrect pcnet32_free_ring
During a code scan for another change I discovered that this call to
pcnet32_free_ring must be removed. If the open fails due to a lack of
memory all the ring structures are removed via the call to free_ring
and a subsequent call to open will dereference a null pointer in
pcnet32_init_ring.
Please apply to 2.6.17.
Signed-off-by: Don Fry <brazilnut@us.ibm.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
| -rw-r--r-- | drivers/net/pcnet32.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/net/pcnet32.c b/drivers/net/pcnet32.c index 07c31f1..fc08c4a 100644 --- a/drivers/net/pcnet32.c +++ b/drivers/net/pcnet32.c @@ -1774,8 +1774,6 @@ static int pcnet32_open(struct net_device *dev) lp->rx_dma_addr[i] = 0; } - pcnet32_free_ring(dev); - /* * Switch back to 16bit mode to avoid problems with dumb * DOS packet driver after a warm reboot |
