On 06/20/2012 01:32 AM, Johannes Berg wrote:
> On Tue, 2012-06-19 at 17:59 -0400, Will Hawkins wrote:
>> Track userspace registrations for authentication
>> frames received on an IBSS interface. This field
>> will be used to decide whether or not to send
>> "open system" authentication frames when a new
>> station joins an adhoc network.
>>
>> Signed-off-by: Will Hawkins <hawkinsw@xxxxxxxxxxxxxxxxxxxxx>
>> ---
>> net/mac80211/cfg.c | 10 ++++++++++
>> net/mac80211/ieee80211_i.h | 1 +
>> 2 files changed, 11 insertions(+)
>>
>> diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c
>> index 498c94e..e00a696 100644
>> --- a/net/mac80211/cfg.c
>> +++ b/net/mac80211/cfg.c
>> @@ -2486,6 +2486,16 @@ static void ieee80211_mgmt_frame_register(struct wiphy *wiphy,
>> u16 frame_type, bool reg)
>> {
>> struct ieee80211_local *local = wiphy_priv(wiphy);
>> + struct ieee80211_sub_if_data *sdata = IEEE80211_DEV_TO_SUB_IF(dev);
>> +
>> + if (sdata->vif.type == NL80211_IFTYPE_ADHOC &&
>> + (frame_type == (IEEE80211_FTYPE_MGMT | IEEE80211_STYPE_AUTH))) {
>> + struct ieee80211_if_ibss *ifibss = &sdata->u.ibss;
>> + if (reg)
>> + ifibss->auth_frame_registrations++;
>> + else if (reg > 0)
>> + ifibss->auth_frame_registrations--;
>
> The "if (reg > 0)" here doesn't seem to make sense?
Just trying to make sure that it doesn't go negative. I am protecting
against unmatched unregisters.
Will
>
> 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
>
--
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
