If ip_randomid() is an asm rather than C code, I have sometimes
seen problems with an asm func calling another asm func.  That
was long ago and far away, but is the only reason I can think of
for that change.

But whether the id is random or a counter, there is no reason to
htons it, as long as it's treated consistently, with externals
never compared with internals.

Barney Wolff

> > Date: Mon, 16 Apr 2001 12:10:19 -0700
> > From: Kris Kennaway <[EMAIL PROTECTED]>
> > 
> > I can't think of anything off the top of my head, but there was some
> > reason why OpenBSD made this change:
> > 
> > -       ip_copy->ip_id = htons(ip_randomid());
> > +       ip_copy->ip_id = ip_randomid();
> > +       HTONS(ip_copy->ip_id);
> > 
> > Presumably there was some reasoning there.  Niels, can you shed any
> > light?

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-net" in the body of the message

Reply via email to