|
Message-ID: <20120619131130.GC7123@cmpxchg8b.com> Date: Tue, 19 Jun 2012 15:11:30 +0200 From: Tavis Ormandy <taviso@...xchg8b.com> To: john-dev@...ts.openwall.com Subject: Re: [patch] optional new raw sha1 implemetation On Tue, Jun 19, 2012 at 05:03:44PM +0400, Solar Designer wrote: > > I noticed an easy way to fix it is just to check if get_hash() == > > binary_hash() first > > This sounds very wrong to me. When get_hash() is worthwhile to use (as > determined by the thresholds), John will use it instead of cmp_all() on > its own - with no magic needed on your part. > > Perhaps there's some context to this that I am missing, though? > I don't think so, the performance impact is indeed very small, 0.1% sounds right. Still, I want that 0.1% :-) Tavis. -- ------------------------------------- taviso@...xchg8b.com | pgp encrypted mail preferred -------------------------------------------------------
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.