Ethereal-dev: Re: [ethereal-dev] prelim support for NTP

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Guy Harris <guy@xxxxxxxxxx>
Date: Wed, 13 Oct 1999 22:26:24 -0700 (PDT)
> This adds initial dissector support for NTP. Still need to add a
> bitfield for the flags, and possibly further process the floating point
> fields, but this is a start. At the moment, everything is just split
> into bytes.
> 
> A patch and two files are attached.

Checked in.

> Index: packet-tcp.c
> ===================================================================
> RCS file: /cvsroot/ethereal/packet-tcp.c,v
> retrieving revision 1.35
> diff -u -r1.35 packet-tcp.c
> --- packet-tcp.c	1999/10/14 01:28:26	1.35
> +++ packet-tcp.c	1999/10/14 04:25:17
> @@ -76,6 +76,7 @@
>  #define TCP_PORT_HTTP     80
>  #define TCP_PORT_POP      110
>  #define TCP_PORT_NNTP     119
> +#define TCP_PORT_NTP      123
>  #define TCP_PORT_NBSS     139
>  #define TCP_PORT_PRINTER  515
>  #define TCP_ALT_PORT_HTTP 8080
> @@ -485,8 +486,10 @@
>        pi.match_port = TCP_PORT_POP;
>        dissect_pop(pd, offset, fd, tree);
>      } else if (PORT_IS(TCP_PORT_NNTP)) {
> -      pi.match_port = TCP_PORT_NNTP;

Nope, the NNTP dissector uses that to decide whether something's an NNTP
request or response.

> Index: packet.h
> ===================================================================
> RCS file: /cvsroot/ethereal/packet.h,v
> retrieving revision 1.107
> diff -u -r1.107 packet.h
> --- packet.h	1999/10/14 01:28:28	1.107
> +++ packet.h	1999/10/14 04:25:22
> @@ -270,6 +270,8 @@
>  	ETT_TELNET,
>  	ETT_TELNET_SUBOPT,
>  	ETT_NNTP,
> +	ETT_NTP,
> +	ETT_NTP_FLAGS,
>  	ETT_SNMP,
>  	ETT_NBSS,
>  	ETT_NBSS_FLAGS,

I also added a declaration of "dissect_ntp()" - the dissector(s)
exported by a "packet-XXX.c" file should be declared in "packet.h".