Wireshark-commits: [Wireshark-commits] master edcd2dd: NTP: fix parsing of control assignments with

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: Thu, 28 Aug 2014 07:14:59 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=edcd2dd57f1649dbb25f1ad0f4a3a77c4d454322
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

edcd2dd by Pascal Quantin (pascal.quantin@xxxxxxxxx):

    NTP: fix parsing of control assignments with empty value
    
    Consider that ',' is a separator between assignments and make value presence optional
    
    Bug: 10417
    Change-Id: I23f2b4029548a1263d65ee11c6356270e7a89bd7
    Reviewed-on: https://code.wireshark.org/review/3884
    Reviewed-by: Evan Huus <eapache@xxxxxxxxx>
    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  14b0d71   RPKI-RTR: enhance dissector
    adds  edcd2dd   NTP: fix parsing of control assignments with empty value


Summary of changes:
 epan/dissectors/packet-ntp.c |    9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)