Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 23 Jul 2002 14:05:42 -0700
From:      Peter Wemm <peter@wemm.org>
To:        "Crist J. Clark" <cjc@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/lib/libstand in_cksum.c 
Message-ID:  <20020723210542.4E2303808@overcee.wemm.org>
In-Reply-To: <20020723204210.GB56146@blossom.cjclark.org> 

next in thread | previous in thread | raw e-mail | index | archive | help
"Crist J. Clark" wrote:
> On Mon, Jul 22, 2002 at 02:28:50AM -0700, Peter Wemm wrote:
> > peter       2002/07/22 02:28:50 PDT
> > 
> >   Modified files:        (Branch: RELENG_4)
> >     lib/libstand         in_cksum.c 
> >   Log:
> >   MFC: rev 1.4:  Make in_cksum() work on little-endian machines.
> >   
> >   Revision     Changes    Path
> >   1.1.1.1.6.1  +20 -5     src/lib/libstand/in_cksum.c
> 
> Err... The checksum algorithm should not be endian dependent, no?
> Isn't that the whole point of the ones-complement addition?
> 
> 	0xabcd		0xcdab
> 	0x1234		0x3412
> 	------		------
> 	0xbe01          0x01be

Umm, have you read the code?  Carefully?

Cheers,
-Peter
--
Peter Wemm - peter@wemm.org; peter@FreeBSD.org; peter@yahoo-inc.com
"All of this is for nothing if we don't go to the stars" - JMS/B5


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020723210542.4E2303808>