Wireshark-bugs: [Wireshark-bugs] [Bug 8490] Dissector for NASDAQ's OUCH 4.x protocol
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Mon, 18 Mar 2013 01:54:06 +0000

changed bug 8490

What Removed Added
Status UNCONFIRMED INCOMPLETE
CC   eapache@gmail.com
Ever confirmed   1

Comment # 1 on bug 8490 from 
Many similar comments to the SoupBinTCP dissector. In addition:

- val_to_str uses the "Unknown" string as a format string passing a uint. You
can either put a %u in, so something like "Unknown (%u)" or you can just use
val_to_str_const which simply returns the string as-is without formatting it.

- you should likely use tvb_reported_length instead of tvb_length so the
dissector behaves correctly when the captured packet has been truncated

- by convention proto_register_* is near the bottom, so should be below
dissect_ouch_heur (though this is very minor)

- rather than putting in (void)data; statements to avoid unused-parameter
warnings, mark them explicitly unused with the _U_ flag that wireshark defines
when building, so something like int myFunc(int used, int unused _U_, int used)

Some of these may apply to SoupBinTCP as well, I just didn't notice them there.

In general both dissectors look pretty nice, they just need a bit of work (and
we need to document our APIs a little better!)

Evan

You are receiving this mail because:
  • You are watching all bug changes.