On Sun, Oct 14, 2012 at 10:20 PM, Jakub Zawadzki
<[email protected]>wrote:

> On Fri, Oct 12, 2012 at 02:02:53PM +0000, Anders Broman wrote:
> > Why not make a patch that developers can try out and start debugging?
>
> Attached.
>
> PS. I'd rather want to hear comments, rather than debugging...
>
>
I would be in favour of having having the trunk build fail (hopefully with
a useful warning) whenever someone accesses ep memory after dissection of
the packet has finished.  Hopefully there won't be too many places where it
is still being accessed late and problems can be fixed quickly.  The only
place I've seen so far is tcp_graph.c (and rlc_lte_graph.c, which had
copied the same approach to identifying the stream/channel by dissecting
just the currently selected frame).  I did review a couple of other taps
but didn't see the same problem.

Martin
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:[email protected]?subject=unsubscribe

Reply via email to