Drivers that require beacon templates will also have the
control structure at their disposal and should always free it.

bcm43xx doesn't use the control structure, but should still free it.

Signed-off-by Ivo van Doorn <[EMAIL PROTECTED]>

---

diff --git a/drivers/net/wireless/d80211/bcm43xx/bcm43xx_main.c 
b/drivers/net/wireless/d80211/bcm43xx/bcm43xx_main.c
index 9f4d51d..24be8f5 100644
--- a/drivers/net/wireless/d80211/bcm43xx/bcm43xx_main.c
+++ b/drivers/net/wireless/d80211/bcm43xx/bcm43xx_main.c
@@ -2732,8 +2732,10 @@ static int bcm43xx_config_interface(struct ieee80211_hw 
*hw,
                if (bcm43xx_is_mode(wl, IEEE80211_IF_TYPE_AP)) {
                        assert(conf->type == IEEE80211_IF_TYPE_AP);
                        bcm43xx_set_ssid(dev, conf->ssid, conf->ssid_len);
-                       if (conf->beacon)
+                       if (conf->beacon) {
                                bcm43xx_refresh_templates(dev, conf->beacon);
+                               kfree(conf->beacon_control);
+                       }
                }
        }
        spin_unlock_irqrestore(&wl->irq_lock, flags);
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to