[PATCH] libertas: don't tear down netdev in libertas_activate_card
authorDan Williams <dcbw@redhat.com>
Fri, 25 May 2007 17:36:57 +0000 (13:36 -0400)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 11 Jun 2007 18:28:41 +0000 (14:28 -0400)
libertas_activate_card() doesn't create the netdev, and shouldn't
free it on error.  The caller of libertas_activate_card() is
responsible for cleaning up errors from libertas_add_card(),
not libertas_activate_card().

Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/libertas/main.c

index b05a6f99a1b29cffbe67dc77f45ab13e85f60e5c..5fde636d1863e932b87525f49b519e47ef68e0ac 100644 (file)
@@ -878,8 +878,6 @@ err_registerdev:
        /* Stop the thread servicing the interrupts */
        wake_up_interruptible(&priv->mainthread.waitq);
        wlan_terminate_thread(&priv->mainthread);
-       kfree(priv->adapter);
-       free_netdev(dev);
 done:
        lbs_deb_leave_args(LBS_DEB_NET, "ret %d", ret);
        return ret;