Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 8db3271c authored by Vasiliy Kulikov's avatar Vasiliy Kulikov Committed by Greg Kroah-Hartman
Browse files

staging: ath6kl: use free_netdev(netdev) instead of kfree() 



It is not guaranteed that free_netdev() is macro.
Freeing netdev without free_netdev() leads to net, tx leaks.
I might lead to dereferencing freed pointer.

Signed-off-by: default avatarVasiliy Kulikov <segooon@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 2166abeb

drivers/staging/ath6kl/os/linux/ar6000_drv.c

+0 −8
Original line number Diff line number Diff line
@@ -2143,11 +2143,7 @@ ar6000_destroy(struct net_device *dev, unsigned int unregister) 
        unregister_netdev(dev);

        is_netdev_registered = 0;

    }

#ifndef free_netdev

    kfree(dev);

#else

    free_netdev(dev);

#endif



#ifdef ATH6K_CONFIG_CFG80211

    ar6k_cfg80211_deinit(ar);
@@ -6429,11 +6425,7 @@ A_STATUS ar6000_remove_ap_interface(AR_SOFTC_T *ar) 
        ar6000_stop_ap_interface(ar);



        unregister_netdev(arApNetDev);

#ifndef free_netdev

        kfree(arApNetDev);

#else

        free_netdev(apApNetDev);

#endif



        A_PRINTF("Remove AP interface\n");

    }