cfg80211: remove superfluous !last_request check in reg_device_remove()
authorChris Wright <chrisw@sous-sol.org>
Fri, 24 Apr 2009 21:09:31 +0000 (14:09 -0700)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 6 May 2009 19:14:40 +0000 (15:14 -0400)
Commit 0ad8acaf "cfg80211: fix NULL pointer deference in
reg_device_remove()" added a check that last_request is non-NULL,
rendering the 2nd check superfluous.  While there, rearrange the code a
bit so it's a little more straight forward.

Signed-off-by: Chris Wright <chrisw@sous-sol.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/wireless/reg.c

index f38cc39fa79e93d2261ababc01dc4c76b611fa47..9fea910204db19371b700ecba069061e7250e8ab 100644 (file)
@@ -2100,14 +2100,14 @@ void reg_device_remove(struct wiphy *wiphy)
 
        assert_cfg80211_lock();
 
+       kfree(wiphy->regd);
+
        if (last_request)
                request_wiphy = wiphy_idx_to_wiphy(last_request->wiphy_idx);
 
-       kfree(wiphy->regd);
-       if (!last_request || !request_wiphy)
-               return;
-       if (request_wiphy != wiphy)
+       if (!request_wiphy || request_wiphy != wiphy)
                return;
+
        last_request->wiphy_idx = WIPHY_IDX_STALE;
        last_request->country_ie_env = ENVIRON_ANY;
 }