Wireshark · Wireshark-bugs: [Wireshark-bugs] [Bug 8691] Adding support of BGP flow spec RFC 5575

[bugzilla-daemon@xxxxxxxxxxxxx]

Comment # 7 on bug 8691 from Matt Texier

Hi Alexis,

Thanks for driving me into the normal process of proposing a new extension to
Wireshark. I am running the various script including fuzzed and it doesn't look
not too bad ... at least for the moment :-) !

With regards to the use of add_item vs add_text and getting back to my code, I
do remember why I did it this way: the size of each value field of the BGP flow
spec NLRI is a variable size. You need first to decode the size of the value
(amount of bytes) and then decode the value. I didn't find the way to register
an hf with a size which is not fixed. I workaround the issue by using a
add_text with a tvb_get_guint8 or tvb_get_ntohs function depending of value
size ... I knew there were something nicer to achieve this but I didn't found
the way !

Can I ask you to point me to an example ? 

Thanks again,
Matt.

---

      You are receiving this mail because:

• You are watching all bug changes.