mac80211: clean up an ieee80211_do_open error path
authorJohannes Berg <johannes.berg@intel.com>
Tue, 3 Apr 2012 12:45:54 +0000 (14:45 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 11 Apr 2012 20:23:51 +0000 (16:23 -0400)
Eliad's comment prompted me to look closer at
the error paths in ieee80211_do_open() and I
found one that should use the error labels.

Also add a comment about the clear_bit since
in many error cases the bit hasn't been set.

Cc: Eliad Peller <eliad@wizery.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/mac80211/iface.c

index ed297649c5773b3db3b0921677e94a9328d64e32..6e85faed053d83afa375efe544ea96083be7ed05 100644 (file)
@@ -349,9 +349,8 @@ static int ieee80211_do_open(struct net_device *dev, bool coming_up)
                memcpy(dev->perm_addr, dev->dev_addr, ETH_ALEN);
 
                if (!is_valid_ether_addr(dev->dev_addr)) {
-                       if (!local->open_count)
-                               drv_stop(local);
-                       return -EADDRNOTAVAIL;
+                       res = -EADDRNOTAVAIL;
+                       goto err_stop;
                }
        }
 
@@ -485,6 +484,7 @@ static int ieee80211_do_open(struct net_device *dev, bool coming_up)
        sdata->bss = NULL;
        if (sdata->vif.type == NL80211_IFTYPE_AP_VLAN)
                list_del(&sdata->u.vlan.list);
+       /* might already be clear but that doesn't matter */
        clear_bit(SDATA_STATE_RUNNING, &sdata->state);
        return res;
 }