Wireshark-commits: [Wireshark-commits] master-1.12 76c93e1: Cleanup a bunch of 802.11ad stuff.
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 6 Aug 2015 02:01:13 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=76c93e1fd8394a388dbe2b3532922583e288f77d
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark

Commits:

76c93e1 by Guy Harris (guy@xxxxxxxxxxxx):

    Cleanup a bunch of 802.11ad stuff.
    
    Make all the multi-byte fields, except for OUIs, little-endian -
    including fields that contain bitfields.  Make the bits in the bitfields
    little-endian as well.
    
    For the "number of taps" bitfields, interpret the values.
    
    Fix a typo ("pseduo" -> "pseudo").
    
    The length of an OCT MMPDU is 16 bits, not 8 bits, and it's not a text
    string, it's an octet string.
    
    Handle the beam refinement UI as a 2-byte field that overlaps with a
    4-byte field - *none* of the bitfields align on nice 8-bit boundaries,
    so that's the best we can do.
    
    Bug: 11419
    Change-Id: Ie2d8b1224783a0bef975dcd66f93ce72b35ba9f4
    Reviewed-on: https://code.wireshark.org/review/9887
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  0a3e315   MAC CTRL: fix dissection of PAUSE frame
    adds  76c93e1   Cleanup a bunch of 802.11ad stuff.


Summary of changes:
 epan/dissectors/packet-ieee80211.c |  400 +++++++++++++++++++-----------------
 1 file changed, 211 insertions(+), 189 deletions(-)