Wireshark-commits: [Wireshark-commits] master a2cbec7: nordic_ble: Fix delta time start-to-start ca
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 13 Mar 2018 11:41:29 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a2cbec743e9c25154281413ea6062ecf81a8a5ea
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

a2cbec7 by Stig Bjørlykke (stig@xxxxxxxxxxxxx):

    nordic_ble: Fix delta time start-to-start calculation
    
    The on-air time for a packet was calculated wrong because it was
    using the wrong number of on-air bytes.  This has been corrected.
    
    Calculating delta time start-to-start for BLE 4.2 requires support
    for on-air packet size up to 255 bytes (LE Data Length Extensions).
    For this the payload length size in the header has changed to two
    bytes.
    
    Calculating delta time start-to-start for BLE 5.0 needs physical
    layer data rate (LE 1M PHY or LE 2M PHY). For this the flags was
    extended with PHY values.
    
    Removed superfluous min and max length checks.
    
    Change-Id: I40bef14f0c19ee77a402efc76e5d01826e63e603
    Reviewed-on: https://code.wireshark.org/review/26457
    Petri-Dish: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  81068f9   GSM BSSMAP: fix a typo in in speech version range_string
    adds  a2cbec7   nordic_ble: Fix delta time start-to-start calculation


Summary of changes:
 epan/dissectors/packet-nordic_ble.c | 279 +++++++++++++++++++++++++-----------
 1 file changed, 199 insertions(+), 80 deletions(-)