lkml.org 
[lkml]   [2011]   [Dec]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[089/104] cfg80211: amend regulatory NULL dereference fix
    3.1-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: "Luis R. Rodriguez" <mcgrof@qca.qualcomm.com>

    commit 0bac71af6e66dc798bf07d0c0dd14ee5503362f9 upstream.

    Johannes' patch for "cfg80211: fix regulatory NULL dereference"
    broke user regulaotry hints and it did not address the fact that
    last_request was left populated even if the previous regulatory
    hint was stale due to the wiphy disappearing.

    Fix user reguluatory hints by only bailing out if for those
    regulatory hints where a request_wiphy is expected. The stale last_request
    considerations are addressed through the previous fixes on last_request
    where we reset the last_request to a static world regdom request upon
    reset_regdomains(). In this case though we further enhance the effect
    by simply restoring reguluatory settings completely.

    Cc: Johannes Berg <johannes.berg@intel.com>
    Signed-off-by: Luis R. Rodriguez <mcgrof@qca.qualcomm.com>
    Reviewed-by: Johannes Berg <johannes@sipsolutions.net>
    Signed-off-by: John W. Linville <linville@tuxdriver.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    net/wireless/reg.c | 6 ++++--
    1 file changed, 4 insertions(+), 2 deletions(-)

    --- a/net/wireless/reg.c
    +++ b/net/wireless/reg.c
    @@ -2040,8 +2040,10 @@ static int __set_regdom(const struct iee
    }

    request_wiphy = wiphy_idx_to_wiphy(last_request->wiphy_idx);
    - if (!request_wiphy) {
    - reg_set_request_processed();
    + if (!request_wiphy &&
    + (last_request->initiator == NL80211_REGDOM_SET_BY_DRIVER ||
    + last_request->initiator == NL80211_REGDOM_SET_BY_COUNTRY_IE)) {
    + schedule_delayed_work(&reg_timeout, 0);
    return -ENODEV;
    }




    \
     
     \ /
      Last update: 2011-12-07 17:37    [W:4.227 / U:0.192 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site