[PATCH] pcnet32: remove incorrect pcnet32_free_ring
authorDon Fry <brazilnut@us.ibm.com>
Thu, 25 May 2006 23:22:40 +0000 (16:22 -0700)
committerJeff Garzik <jeff@garzik.org>
Sat, 27 May 2006 01:33:18 +0000 (21:33 -0400)
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>
drivers/net/pcnet32.c

index 07c31f19c6ba40d419b2d022d069af321130f73f..fc08c4af506ca8194b7f4d3854181fdc5eed43b4 100644 (file)
@@ -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