Re: dissection question

[Guy Harris <guy () alum mit edu>]

On Feb 16, 2010, at 11:18 AM, Brian Oleksa wrote:

>                    guint8 sizeofroutername;

That should presumably be

                   guint16 sizeofroutername;

as, according to

        https://www.darkcornersoftware.com/confluence/display/open/Minotaur+SA+ROUTER+Plugin+Extension

it's a 2-byte field, and...

>                        //Size of router name
>                        sizeofroutername = tvb_get_guint8(tvb, offset);

...that should presumably be tvb_get_ntohs() for the same reason.

If you fix that, then I suspect that

>                        //Router Name
>                        proto_tree_add_item(helen_sub_tree, hf_helen_routername, tvb, offset, sizeofroutername, FALSE);
>                        offset += sizeofroutername;

will work.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe