[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.026 / U:1.772 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site