Wireshark · Ethereal-dev: Re: [Ethereal-dev] ISUP patch

[Guy Harris <guy@xxxxxxxxxxxx>]

On Fri, Aug 15, 2003 at 11:31:37AM -0700, Michael Lum wrote:
> I believe the ISUP dissector has a bug when decoding the
> optional parameter pointer.

So what is the added check doing?  It looks as if it's checking whether
the optional parameter pointer is present in the packet.

> Please see the attached isup.snoop.  Packet 5 contains an RLC in
> which the last octet (optional parameter pointer) is not handled
> correctly.

Unfortunately, as per your mail, that's a trace from a machine running
code that implements an old M2UA draft (older than even the draft
implemented by the first version of the M2UA dissector in Ethereal), so
if it shows ISUP traffic, you presumably have a dissector that you've
modified to support that older draft - should a dissector preference be
added to allow the RFC, and various older drafts, to be supported?