[Ieee80211-devel] bogus code (or comment) wrt. rx_stats

[Johannes B. <joh...@si...>]

There's a comment in the header file saying:

/* NOTE: This data is for statistical purposes; not all hardware provides t=
his
 *       information for frames received.  Not setting these will not cause
 *       any adverse affects. */

Now, this is false. As it happens, both ieee80211_handle_assoc_resp and
ieee80211_network_init rely on stats->len being correct while calling
ieee80211_parse_info_param, which in turn are called if one calls
ieee80211_rx_mgt.

It seems that ieee80211_rx_mgt should have a length parameter and pass
that down (probably instead of the stats, since all those functions
don't really need them), but fixing the comment would be good enough I
guess.

Please fix either the comment or the code.

johannes