Wireshark · Ethereal-dev: Re: [Ethereal-dev] Re: New dissectors: H.223 and friends

[Guy Harris <gharris@xxxxxxxxx>]

Richard van der Hoff wrote:

> yup, that seems fine. Sorry again for overlooking these - it's quite 
> hard to spot them with a compiler that doesn't complain :(.

There's an assertion violation - it was causing a crash when the 
assertion was done with g_assert(), but dissectors should use 
DISSECTOR_ASSERT() so that, while dissection of the packet that provokes 
the dissector bug is aborted, procesing the capture file isn't.  I've 
checked in a change to make it do so.

See bug 508:

	http://bugs.ethereal.com/bugzilla/show_bug.cgi?id=508

which has a capture file that provokes the problem attached.