On Oct 10, 2011, at 4:34 PM, wmeier@xxxxxxxxxxxxx wrote:
> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=39350
>
> User: wmeier
> Date: 2011/10/10 04:34 PM
>
> Log:
> Fix bugs which caused certain fields to display with the wrong endianness;
> Specifically: some proto_tree_add_item() 'encoding' args were incorrect.
A lot of these bugs appear to be the result of developers typing "item" when they mean "uint" - i.e., they extracted a field from the packet in order to use its value later in the dissection, and then, in order to put it into the protocol tree without re-fetching it, they intended to do
proto_tree_add_uint(..., value);
but instead typed
proto_tree_add_item(..., value);
(Another reason why I'm more and more inclined to believe that the current way dissectors are written is much too low-level.)