[lkml]   [1997]   [Apr]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: WARNING: that patch from Ted...
Er, um, I'm *really* sorry to point out bugs twice in a row, but when
merging our patches, you made a mistake. To be precise, the
+ seq = (halfMD4Transform(secret+8, secret) &
+ ((1<<HASH_BITS)-1)) + (count << HASH_BITS);
part. I left the << HASH_BITS out because I changed the code so count
was a __u32, pre-shifted. Thus, you don't *want* the count, this
shift makes the result always 0, and the ISN will thus occasionally step
backwards, a not-so-wonderful thing.

Sorry, I should have made that little speed optimization (this whole patch
is a speed optimization, so it seemed apropos) clearer when I sent it to

 \ /
  Last update: 2005-03-22 13:39    [W:0.022 / U:14.468 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site