Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 52578: /trunk/epan/ /trunk/epan/: pr
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Jakub Zawadzki <darkjames-ws@xxxxxxxxxxxx>
Date: Mon, 14 Oct 2013 07:55:18 +0200
On Sun, Oct 13, 2013 at 04:43:28PM -0400, Evan Huus wrote:
> On Sun, Oct 13, 2013 at 3:54 AM, Jakub Zawadzki
> <darkjames-ws@xxxxxxxxxxxx> wrote:
> > About tvb_offset_exists() comment, compute_offset() is not returning
> > exception when offset == tvb->length.
> 
> Ah, OK. Should it? When offset == tvb->length I would think that
> should be an exception, since no bytes exist at that location.

Don't know, right now it works :}

There is some lengthy comment about it before check_offset_length_no_exception().

I know about some dissectors which calls tvb_find_guint8() with offset == tvb->offset [bug #9034#c5].