On Mon, 2008-04-21 at 13:24 +0800, Zhu Yi wrote: > On Sun, 2008-04-20 at 19:36 -0700, Harvey Harrison wrote: > > - hdrlen += (fc & IEEE80211_STYPE_QOS_DATA) > > - >> (ilog2(IEEE80211_STYPE_QOS_DATA)-1); > > + if (fc & IEEE80211_STYPE_QOS_DATA) > > + hdrlen += 2; > > You need also take other QoS types (i.e QOS_NULLFUNC) into > consideration. Please take a look at WLAN_FC_IS_QOS_DATA macro. > Well, that would be a change from what's there now, I've just got a few pure cleanup patches I've done while making some changes in wpa.c I'm going to submit the cleanup patches on their own first, then patches with more substance. Cheers, Harvey -- 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