Wireshark-bugs: [Wireshark-bugs] [Bug 4753] New dissector: JSON
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Fri, 14 May 2010 13:02:19 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4753

--- Comment #2 from Chris Maynard <christopher.maynard@xxxxxxxxx> 2010-05-14 13:02:17 PDT ---
Some initial feedback, tested against r32806:

1) $ tools/checkhf.pl epan/dissectors/packet-json.c
Unused entry: epan/dissectors/packet-json.c, hf_json_member_key

2) Forgetting Frame #2, only Frame #1 appears to be dissected as json.  I don't
notice any difference in Frames #3 and #4 as compared to Wireshark 1.2.8.

3) Why do frames #3-4 display "(application/json)" twice in the Info column?
4) Any XXX's need further attention?
5) fuzz-testing?
6) Instead of just indicating RFC 4627, maybe you could add a link to it, such
as:
http://www.ietf.org/rfc/rfc4627.txt?number=4627

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.