On Tue, Apr 27, 2010 at 04:39:37PM +0200, Joakim Tjernlund wrote:
> > > I think I mentioned this in passing but the fletcher checksum impl.
> > > is fragile. Try changing "u16 length" to "u32 length" and watch it
> > > break for no apparent reason.
> >
> > Yes, i noticed that and will fix that. Thanks.
> 
> You know what the fix is? a (int) type cast like so:
>   x = (int)((length - LSA_CHECKSUM_OFFSET) * c0 - c1) % 255;

Yes, i noticed that.

-- 
Elen sila lumenn' omentielvo

Ondrej 'SanTiago' Zajicek (email: santi...@crfreenet.org)
OpenPGP encrypted e-mails preferred (KeyID 0x11DEADC3, wwwkeys.pgp.net)
"To err is human -- to blame it on a computer is even more so."

Attachment: signature.asc
Description: Digital signature

Reply via email to