ipw2200: fix oops on missing firmware
For non-monitor interfaces, the syntax for alloc_ieee80211/free_80211 is wrong. Because alloc_ieee80211 only creates (wiphy_new) a wiphy, but free_80211() does wiphy_unregister() also. This is only correct when the later wiphy_register() is called successfully, which apparently is not the case for your fw doesn't exist one. Signed-off-by: Zhu Yi <yi.zhu@intel.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
parent
63ca2d74ea
commit
e6c5fc53d0
4 changed files with 16 additions and 6 deletions
|
@ -6325,8 +6325,10 @@ static int ipw2100_pci_init_one(struct pci_dev *pci_dev,
|
||||||
|
|
||||||
fail:
|
fail:
|
||||||
if (dev) {
|
if (dev) {
|
||||||
if (registered)
|
if (registered) {
|
||||||
|
unregister_ieee80211(priv->ieee);
|
||||||
unregister_netdev(dev);
|
unregister_netdev(dev);
|
||||||
|
}
|
||||||
|
|
||||||
ipw2100_hw_stop_adapter(priv);
|
ipw2100_hw_stop_adapter(priv);
|
||||||
|
|
||||||
|
@ -6383,6 +6385,7 @@ static void __devexit ipw2100_pci_remove_one(struct pci_dev *pci_dev)
|
||||||
/* Unregister the device first - this results in close()
|
/* Unregister the device first - this results in close()
|
||||||
* being called if the device is open. If we free storage
|
* being called if the device is open. If we free storage
|
||||||
* first, then close() will crash. */
|
* first, then close() will crash. */
|
||||||
|
unregister_ieee80211(priv->ieee);
|
||||||
unregister_netdev(dev);
|
unregister_netdev(dev);
|
||||||
|
|
||||||
/* ipw2100_down will ensure that there is no more pending work
|
/* ipw2100_down will ensure that there is no more pending work
|
||||||
|
|
|
@ -11821,6 +11821,7 @@ static int __devinit ipw_pci_probe(struct pci_dev *pdev,
|
||||||
if (err) {
|
if (err) {
|
||||||
IPW_ERROR("Failed to register promiscuous network "
|
IPW_ERROR("Failed to register promiscuous network "
|
||||||
"device (error %d).\n", err);
|
"device (error %d).\n", err);
|
||||||
|
unregister_ieee80211(priv->ieee);
|
||||||
unregister_netdev(priv->net_dev);
|
unregister_netdev(priv->net_dev);
|
||||||
goto out_remove_sysfs;
|
goto out_remove_sysfs;
|
||||||
}
|
}
|
||||||
|
@ -11871,6 +11872,7 @@ static void __devexit ipw_pci_remove(struct pci_dev *pdev)
|
||||||
|
|
||||||
mutex_unlock(&priv->mutex);
|
mutex_unlock(&priv->mutex);
|
||||||
|
|
||||||
|
unregister_ieee80211(priv->ieee);
|
||||||
unregister_netdev(priv->net_dev);
|
unregister_netdev(priv->net_dev);
|
||||||
|
|
||||||
if (priv->rxq) {
|
if (priv->rxq) {
|
||||||
|
|
|
@ -1020,6 +1020,7 @@ static inline int libipw_is_cck_rate(u8 rate)
|
||||||
/* ieee80211.c */
|
/* ieee80211.c */
|
||||||
extern void free_ieee80211(struct net_device *dev, int monitor);
|
extern void free_ieee80211(struct net_device *dev, int monitor);
|
||||||
extern struct net_device *alloc_ieee80211(int sizeof_priv, int monitor);
|
extern struct net_device *alloc_ieee80211(int sizeof_priv, int monitor);
|
||||||
|
extern void unregister_ieee80211(struct libipw_device *ieee);
|
||||||
extern int libipw_change_mtu(struct net_device *dev, int new_mtu);
|
extern int libipw_change_mtu(struct net_device *dev, int new_mtu);
|
||||||
|
|
||||||
extern void libipw_networks_age(struct libipw_device *ieee,
|
extern void libipw_networks_age(struct libipw_device *ieee,
|
||||||
|
|
|
@ -235,14 +235,17 @@ void free_ieee80211(struct net_device *dev, int monitor)
|
||||||
libipw_networks_free(ieee);
|
libipw_networks_free(ieee);
|
||||||
|
|
||||||
/* free cfg80211 resources */
|
/* free cfg80211 resources */
|
||||||
if (!monitor) {
|
if (!monitor)
|
||||||
|
wiphy_free(ieee->wdev.wiphy);
|
||||||
|
|
||||||
|
free_netdev(dev);
|
||||||
|
}
|
||||||
|
|
||||||
|
void unregister_ieee80211(struct libipw_device *ieee)
|
||||||
|
{
|
||||||
wiphy_unregister(ieee->wdev.wiphy);
|
wiphy_unregister(ieee->wdev.wiphy);
|
||||||
kfree(ieee->a_band.channels);
|
kfree(ieee->a_band.channels);
|
||||||
kfree(ieee->bg_band.channels);
|
kfree(ieee->bg_band.channels);
|
||||||
wiphy_free(ieee->wdev.wiphy);
|
|
||||||
}
|
|
||||||
|
|
||||||
free_netdev(dev);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
#ifdef CONFIG_LIBIPW_DEBUG
|
#ifdef CONFIG_LIBIPW_DEBUG
|
||||||
|
@ -330,3 +333,4 @@ module_init(libipw_init);
|
||||||
|
|
||||||
EXPORT_SYMBOL(alloc_ieee80211);
|
EXPORT_SYMBOL(alloc_ieee80211);
|
||||||
EXPORT_SYMBOL(free_ieee80211);
|
EXPORT_SYMBOL(free_ieee80211);
|
||||||
|
EXPORT_SYMBOL(unregister_ieee80211);
|
||||||
|
|
Loading…
Add table
Reference in a new issue