Wireshark-commits: [Wireshark-commits] master-1.12 9e978d3: Fix an error where we expect the tagged

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 25 Jul 2015 17:39:37 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=9e978d324a0f671cb499d5c3164c1930199c18f9
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark

Commits:

9e978d3 by Richard Sharpe (rsharpe@xxxxxxxxx):

    Fix an error where we expect the tagged parameters field to be present even
    though the spec (802.11ad-2012) says they are optional.
    
    Communicated to me by the WiFi Alliance. I have captures. Not sure if I can
    share them.
    
    Change-Id: Id5998594214ac4b6a1d3baf3cb2f0d4fe6227b40
    Signed-off-by: Richard Sharpe <realrichardsharpe@xxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/9785
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    (cherry picked from commit 47c175376b0abb0ae6ab42e1072c3a6740c7baa4)
    Reviewed-on: https://code.wireshark.org/review/9789
    

Actions performed:

    from  9831d16   Fix ptvcursor_add() so it can dissect the last bytes in a TVB again.
    adds  9e978d3   Fix an error where we expect the tagged parameters field to be present even though the spec (802.11ad-2012) says they are optional.


Summary of changes:
 epan/dissectors/packet-ieee80211.c |   11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)