Wireshark-bugs: [Wireshark-bugs] [Bug 7658] New: LLRP: Dissect parameter fields
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Tue, 21 Aug 2012 13:33:10 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7658

           Summary: LLRP: Dissect parameter fields
           Product: Wireshark
           Version: SVN
          Platform: x86
        OS/Version: Gentoo
            Status: NEW
          Severity: Enhancement
          Priority: Low
         Component: Dissection engine (libwireshark)
        AssignedTo: bugzilla-admin@xxxxxxxxxxxxx
        ReportedBy: magonk@xxxxxxxxx


Created attachment 8991
  --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=8991
patch to LLRP dissector

Build Information:
Paste the COMPLETE build information from "Help->About Wireshark", "wireshark
-v", or "tshark -v".
--
Hi,

I have added support for parameter dissection to the current LLRP dissector.

It supports all the messages and parameters of LLRP protocol version 1.1.
It also supports most of Impinj extension to this protocol.

It validates against checkAPI and also survives for 300 rounds of fuzzy test.

There is a small problem. The patch starts with a modification of
"epan/proto.c".
I believe that there is a bug in the handling of FT_UINT_STRING. I want to pass
"ENC_BIG_ENDIAN | ENC_UTF_8" to proto_tree_add_item(). But without this patch
it is not possible as get_uint_value() will treat my value as little-endian.

There is also workaround using just "ENC_BIG_ENDIAN" in the FT_UINT_STRING
call. This will work as the ENC_UTF_8 flag is not handled anyway.

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