Wireshark-bugs: [Wireshark-bugs] [Bug 8313] new dissector for SML protocol
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Fri, 15 Feb 2013 15:07:25 +0000

Comment # 17 on bug 8313 from 
(In reply to comment #16)
> (In reply to comment #15)
> > Comment on attachment 10017 [details]
> > smb dissector 15.02 (updated)
> > 
> > Not sure if it was fix-encodings-args.pl, but the patch now fails
> > checkAPIs.pl because of the insertion of ENC_BIG_ENDIAN into
> > proto_tree_add_bytes_format/proto_tree_add_bytes.
> 
> Yes, sorry, I should have caught that (it's been hanging around for a few
> iterations of the patch now). Calls to proto_tree_add_bytes and
> proto_tree_add_bytes_format are specifically for cases where the bytes in
> question are

[BUGZILLA FAIL]

not in the packet. Those calls don't take an encoding argument at all - that
argument should be a pointer to the generated bytes. Since I believe the
intention is to point to bytes that are in the packet itself, you should just
be able to replace those calls with generic proto_tree_add_item calls. These
calls will take an encoding arg, but it is find to leave it as ENC_NA since raw
bytes don't have an endianness.

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