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.