Wireshark-commits: [Wireshark-commits] master-1.10 ff525ed: NTP: fix parsing of control assignments

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 28 Aug 2014 17:32:43 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ff525ed70c32f95934fb8dffad7d2b2f0a0afb0f
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master-1.10
Repository: wireshark

Commits:

ff525ed 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>
    (cherry picked from commit edcd2dd57f1649dbb25f1ad0f4a3a77c4d454322)
    Reviewed-on: https://code.wireshark.org/review/3904
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    

Actions performed:

    from  85e18f3   Increase the error buffer size for rpcap.
    adds  ff525ed   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(-)