Ethereal-dev: Re: [ethereal-dev] Mismatch in "port" integer size

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

From: Gilbert Ramirez <gram@xxxxxxxxxx>
Date: Wed, 5 Jan 2000 15:39:34 -0600
On Wed, Jan 05, 2000 at 02:08:18PM -0600, Guy Harris wrote:
> 
> 
> > So, does packet_info use guint32 to support non-tcp/udp "ports"?
> 
> That was the intent - to allow it to handle protocols where the
> equivalent of a transport-layer port (transport layer service access
> point?) could be longer than 16 bits.
> 
> I'd be inclined to turn the port number arguments into "guint32"s - they
> may actually work better than guint16s on many platforms (e.g., on
> Alpha, you can load an aligned 32-bit quantity from memory or store it
> into memory with a single instruction, but, on processors that don't
> have the BWX extensions, you have to do 16-bit or 8-bit accesses with
> multiple instructions; admittedly, they'll be passed in registers on
> Alpha, but "srcport" and "destport" in "packet_info" structures are in
> memory).

Fine by me. I'll make the change, along with a few other changes I'm making
for win32-compilation.

--gilbert