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
|
