Wireshark · Ethereal-dev: Re: [Ethereal-dev] OSI Session dissector in the works

[Guy Harris <guy@xxxxxxxxxxxx>]

On Fri, Sep 05, 2003 at 08:13:48AM +0000, Sid Sid wrote:
> >Have you enabled COTP reassembly?  If not, do so.  If so, then there's a
> >bug in COTP reassembly, which we need to fix.
> >
> AFAIK, there are not separate packet-cotp.c.

No, there isn't, but that's because when Ralf Schneider wrote the CLNP
and COTP stuff he put them in packet-clnp.c.

> Do you mean COTP code inside packet-clnp.c ? There are not COTP parameters I 
> can change from GUI at all.

There are, but only in the current CVS version.

> IMHO,there are not any reassembling code regarding COTP in osi_tp too. Only 
> for CLNP.

Again, that's in the current CVS version.  If you're going to be
developing an OSI session-layer dissector, you should be using the
current CVS version of Ethereal, or should wait for the next release.

> Is it right to have CLNP and COTP code in one module ?

Probably not, but nobody's bothered to remove the COTP/CLTP code from
packet-clnp.c and put it in a packet-ositp.c.