diff options
author | Johannes Berg <johannes@sipsolutions.net> | 2009-03-24 09:35:46 +0100 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2009-03-27 20:13:20 -0400 |
commit | 4bbf4d56583dd52c429d88f43cb614bdbe5deea6 (patch) | |
tree | 7a3f902a08820342254e0d67607fe870b02620b3 /net/wireless/core.h | |
parent | 3832c287f11ba001bbe48e9be8c59cb9f71f6b43 (diff) | |
download | kernel_samsung_smdk4412-4bbf4d56583dd52c429d88f43cb614bdbe5deea6.zip kernel_samsung_smdk4412-4bbf4d56583dd52c429d88f43cb614bdbe5deea6.tar.gz kernel_samsung_smdk4412-4bbf4d56583dd52c429d88f43cb614bdbe5deea6.tar.bz2 |
cfg80211: fix locking in nl80211_set_wiphy
Luis reports that there's a circular locking dependency;
this is because cfg80211_dev_rename() will acquire the
cfg80211_mutex while the device mutex is held, while
this normally is done the other way around. The solution
is to open-code the device-getting in nl80211_set_wiphy
and require holding the mutex around cfg80211_dev_rename
rather than acquiring it within.
Also fix a bug -- rtnl locking is expected by drivers so
we need to provide it.
Reported-by: Luis R. Rodriguez <lrodriguez@atheros.com>
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/wireless/core.h')
-rw-r--r-- | net/wireless/core.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/net/wireless/core.h b/net/wireless/core.h index 97a6fd8..d43daa2 100644 --- a/net/wireless/core.h +++ b/net/wireless/core.h @@ -99,6 +99,9 @@ struct cfg80211_internal_bss { struct cfg80211_registered_device *cfg80211_drv_by_wiphy_idx(int wiphy_idx); int get_wiphy_idx(struct wiphy *wiphy); +struct cfg80211_registered_device * +__cfg80211_drv_from_info(struct genl_info *info); + /* * This function returns a pointer to the driver * that the genl_info item that is passed refers to. |