Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[802.11] Fix maximum packet length
Previously the maximum packet length was computed using an erroneous
understanding of the role of the MIC field in TKIP-encrypted packets.
The field is actually considered to be part of the MSDU (encrypted and
fragmented data), not the MPDU (container for each encrypted
fragment). As such its size does not contribute to cryptographic
overhead outside the data field's size limitations. The net result is
that the previous maximum packet length value was 4 bytes too long;
fix it to the correct value of 2352.

Signed-off-by: Michael Brown <mcb30@etherboot.org>
  • Loading branch information
rwcr authored and Michael Brown committed Aug 8, 2009
1 parent 1e810be commit 0b3c88e
Showing 1 changed file with 13 additions and 5 deletions.
18 changes: 13 additions & 5 deletions src/include/gpxe/ieee80211.h
Expand Up @@ -31,11 +31,19 @@ FILE_LICENCE(GPL2_OR_LATER);
/** Maximum cryptographic overhead before encrypted data */
#define IEEE80211_MAX_CRYPTO_HEADER 8

/** Maximum cryptographic overhead after encrypted data */
#define IEEE80211_MAX_CRYPTO_TRAILER 12
/** Maximum cryptographic overhead after encrypted data
*
* This does not count the MIC in TKIP frames, since that is
* considered to be part of the MSDU and thus contributes to the size
* of the data field.
*
* It @e does count the MIC in CCMP frames, which is considered part
* of the MPDU (outside the data field).
*/
#define IEEE80211_MAX_CRYPTO_TRAILER 8

/** Total maximum cryptographic overhead */
#define IEEE80211_MAX_CRYPTO_OVERHEAD 20
#define IEEE80211_MAX_CRYPTO_OVERHEAD 16

/** Bytes of network-layer data that can go into a regular data frame */
#define IEEE80211_MAX_FRAME_DATA 2296
Expand All @@ -58,10 +66,10 @@ FILE_LICENCE(GPL2_OR_LATER);

/** Maximum combined frame length
*
* The biggest frame will include 32 frame header bytes, 20 bytes of
* The biggest frame will include 32 frame header bytes, 16 bytes of
* crypto overhead, and 2304 data bytes.
*/
#define IEEE80211_MAX_FRAME_LEN 2356
#define IEEE80211_MAX_FRAME_LEN 2352

/** Maximum length of an ESSID */
#define IEEE80211_MAX_SSID_LEN 32
Expand Down

0 comments on commit 0b3c88e

Please sign in to comment.