Wireshark-commits: [Wireshark-commits] master 47c1753: Fix an error where we expect the tagged para
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: Sat, 25 Jul 2015 17:39:22 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=47c175376b0abb0ae6ab42e1072c3a6740c7baa4
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

47c1753 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>
    

Actions performed:

    from  92fa272   ISAKMP: Add 3GPP IMS Options for IKEv2
    adds  47c1753   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(-)