On Wed, 2011-01-05 at 21:17 +0530, Sujith wrote: > From: Sujith Manoharan <Sujith.Manoharan@xxxxxxxxxxx> > > Add two new per-BSS attributes to allow configuration of > HT capabilites and operational parameters by hostapd. Looks fine, but > @@ -2597,6 +2597,12 @@ static int nl80211_set_bss(struct sk_buff *skb, struct genl_info *info) > if (info->attrs[NL80211_ATTR_BSS_HT_OPMODE]) > params.ht_opmode = > nla_get_u16(info->attrs[NL80211_ATTR_BSS_HT_OPMODE]); > + if (info->attrs[NL80211_ATTR_BSS_HT_CAPAB]) > + params.ht_capab = > + nla_get_u16(info->attrs[NL80211_ATTR_BSS_HT_CAPAB]); > + if (info->attrs[NL80211_ATTR_BSS_HT_PARAM]) > + params.ht_param = > + nla_get_u8(info->attrs[NL80211_ATTR_BSS_HT_PARAM]); > > if (!rdev->ops->change_bss) > return -EOPNOTSUPP; This appears to be missing a -1 initialisation? johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html