Wireshark-commits: [Wireshark-commits] master 7573f7d: radiotap: Correct support for 0-length PSDUs

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 10 Sep 2018 07:34:03 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=7573f7dab49043bc6401b97a10439d9b9b6a291a
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

7573f7d by Richard Sharpe (realrichardsharpe@xxxxxxxxx):

    radiotap: Correct support for 0-length PSDUs.
    
    When there is no data, which is indicated by the 0-length PDSU radiotap header,
    there is no more data to dissect, so don't dissect any more as that causes an
    exception.
    
    Change-Id: I284b8128ec309ba26f24a012380d311eb3e48697
    Reviewed-on: https://code.wireshark.org/review/29529
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  017d615   docsis: fixed regression that was introduced by adding FCS check
     add  7573f7d   radiotap: Correct support for 0-length PSDUs.


Summary of changes:
 epan/dissectors/packet-ieee80211-radiotap-defs.h | 2 +-
 epan/dissectors/packet-ieee80211-radiotap.c      | 9 +++++++++
 2 files changed, 10 insertions(+), 1 deletion(-)