Wireshark-bugs: [Wireshark-bugs] [Bug 9158] LLC: Display I/G and C/R bits in subtree/bitwise vie

Date: Thu, 26 Sep 2013 18:57:54 +0000

Comment # 6 on bug 9158 from
(In reply to comment #5)
> (In reply to comment #4)
> > Jeff, I am not sure I understand your suggestion but my natural idea would
> > be to display both bitmasks in the subtree of DSAP/SSAP value, the SAP and
> > the I/G or C/R bit, while not adding any new fields to the top-level tree,
> 
> Yes, that seems reasonable too.  Maybe someone else might have a different
> opinion on the display in the current patch but to me it does "feel" kind of
> incomplete or funny.  But it may just be me.

Jeff,

I will be happy to update my patch in this way. I would like to ask you for a
little guidance, though: the DSAP field is split into <7:SAP><1:G/I> bit
fields, the SSAP field is split into <7:SAP><1:C/R>. If I define a bitmask of
0xF7 in the corresponding hf_ entry to isolate the SAP value, any
proto_tree_add_... function will first shift the SAP value by 1 bit to the
right and only then use this shifted value when displaying it and when
translating it to the protocol name. This is obviously wrong. What I need is to
isolate the SAP value but not shift it during display and conversion to
protocol name. How can I do this?

Thank you!

Best regards,
Peter


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