Wireshark-commits: [Wireshark-commits] master 5a98368: print: don't ignore FT_NONE fields in ek.

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: Sat, 23 Mar 2019 00:05:23 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=5a98368ad865a6b856743e9e33c5bba9fe7dee3e
Submitter: "Peter Wu <peter@xxxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

5a98368 by Dario Lombardo (lomato@xxxxxxxxx):

    print: don't ignore FT_NONE fields in ek.
    
    The 'null' placeholder has been used in such a fields. Otherwise
    the json dumper would complain for the lack of a value, getting
    confused by a double name add without values.
    
    Bug: 15628
    Change-Id: I016325790f8d4a02ed9288225e861ba2d23a82f9
    Reviewed-on: https://code.wireshark.org/review/32532
    Reviewed-by: Dario Lombardo <lomato@xxxxxxxxx>
    Petri-Dish: Dario Lombardo <lomato@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

Actions performed:

    from  ed40d31   json_dumper: flush dumper before printing an error.
     add  5a98368   print: don't ignore FT_NONE fields in ek.


Summary of changes:
 epan/print.c | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)