ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
July 17th, 2024 | 10:00am-11:55am SGT (UTC+8) | Online

Wireshark-commits: [Wireshark-commits] master 2015398: proto.c: proto_registrar_dump_values(): don'

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 25 Nov 2014 18:52:37 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=20153980ebe4a54824dd3558685d1e7994190398
Submitter: Bill Meier (wmeier@xxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

2015398 by Bill Meier (wmeier@xxxxxxxxxxx):

    proto.c: proto_registrar_dump_values(): don't skip "same name" fields.
    
    It appears that Wireshark allows (and currently has) fields with the
    same name with differing "strings" content.
    
    Change-Id: Iddac5a2c9f456a97ea676f2007f43d786482eebd
    Reviewed-on: https://code.wireshark.org/review/5491
    Reviewed-by: Bill Meier <wmeier@xxxxxxxxxxx>
    

Actions performed:

    from  152b0c9   Remove support for pinfo->private_data in "data-text-lines", "data-l1-events" and "urlencoded-form" dissectors.  Data must be passed from parent through dissectors data parameter.
    adds  2015398   proto.c: proto_registrar_dump_values(): don't skip "same name" fields.


Summary of changes:
 epan/proto.c |  217 +++++++++++++++++++++++++++++-----------------------------
 1 file changed, 108 insertions(+), 109 deletions(-)