Wireshark-commits: [Wireshark-commits] master 70608ef: TWAMP: Handle short TWAMP-Test packets
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: Mon, 19 Feb 2018 15:49:06 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=70608effe4d23bb5f05dc783b21c1ab9d2e5aa66
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

70608ef by Valentin Vidic (Valentin.Vidic@xxxxxxxxx):

    TWAMP: Handle short TWAMP-Test packets
    
    Requests packets have a minimum length of 14 bytes and only
    responses have additional 27 bytes of fields.  Without this
    patch short packets generate an exception and get reported
    as malformed.
    
    Change-Id: If75fa5556059b13f40fc49273edcbd32508fa0fc
    Reviewed-on: https://code.wireshark.org/review/25897
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  8da569b   WSDG: Update docbook info
    adds  70608ef   TWAMP: Handle short TWAMP-Test packets


Summary of changes:
 epan/dissectors/packet-twamp.c | 31 +++++++++++++++++--------------
 1 file changed, 17 insertions(+), 14 deletions(-)