Wireshark-dev: Re: [Wireshark-dev] Proper use of proto_tree_add_string?

From: "Martin Mathieson" <martin.r.mathieson@xxxxxxxxxxxxxx>
Date: Fri, 29 Jun 2007 17:50:05 +0100
You can always specify the offset and length as 0, 0 - i.e. no bytes
will be highlighted when the item is selected.

On 6/29/07, kannoll <maillistnoll@xxxxxxxxxxxxx> wrote:

That's a good idea, and I'll probably have a place to use it, but my
immediate problem is packets that end before expected, so there's nothing to
highlight in the data (because it's not there).

--kan--

-----Original Message-----
From: wireshark-dev-bounces@xxxxxxxxxxxxx
[mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] On Behalf Of Martin Mathieson
Sent: Friday, June 29, 2007 12:39 PM
To: Developer support list for Wireshark
Subject: Re: [Wireshark-dev] Proper use of proto_tree_add_string?

What about doing something like this (taken from the packet-cops.c) ?

      proto_item *ti;
      ti = proto_tree_add_text(tree, tvb, offset, len,
                                            "PEP Id is not a NULL terminated
ASCII string");
      expert_add_info_format(pinfo, ti, PI_MALFORMED, PI_NOTE,
                                           "PEP Id is not a NULL terminated
ASCII string");

This highlights the section of packet data where the problem lies, but
displays a string not taken from the packet data.  It also adds an expert
item, which makes the item easier to navigate to.

Hope this helps,
Martin

On 6/29/07, Jaap Keuter <jaap.keuter@xxxxxxxxx> wrote:
> How about: proto_tree_add_debug_text() ?
>
>
> Thanx,
> Jaap
>
> kannoll wrote:
> >
> >
> > I am running into some error conditions in decoding WLCCP and would
> > like
> to
> > catch the error and print a nice message in the protocol tree at the
> > point the error occurs and stop decoding at that point.
> >
> > I have the logic down, but I'm trying to understand the best way to
> > place the "nice message" into the protocol tree.
> > Proto_tree_add_string sounds
> like
> > the right function, but the arguments seem to indicate otherwise. I
> > think proto_tree_add_string is going to add a string to the tree
> > based on data that is pulled from the packet.
> >
> > In my case, though, I'm not basing my string on data that's in the
packet.
> > Instead, it's just a nice warning message that something is wrong
> > with the packet format.
> >
> > What's the best way to handle this?
>
> _______________________________________________
> Wireshark-dev mailing list
> Wireshark-dev@xxxxxxxxxxxxx
> http://www.wireshark.org/mailman/listinfo/wireshark-dev
>
_______________________________________________
Wireshark-dev mailing list
Wireshark-dev@xxxxxxxxxxxxx
http://www.wireshark.org/mailman/listinfo/wireshark-dev

_______________________________________________
Wireshark-dev mailing list
Wireshark-dev@xxxxxxxxxxxxx
http://www.wireshark.org/mailman/listinfo/wireshark-dev