[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <45d431f42768ab4b4f5b8b02300346d3@smtp.hushmail.com>
Date: Fri, 22 Jun 2012 22:08:12 +0200
From: magnum <john.magnum@...hmail.com>
To: "john-dev@...ts.openwall.com" <john-dev@...ts.openwall.com>
Subject: Re: salts to use when benchmarking (was: SSH thread-safety)
Dhiru, or someone else, please commit this with --author="JimF ..." (or whatever the syntax) and so on. I'll be back in a couple of days at best.
magnum
On 22 jun 2012, at 19:39, "jfoug" <jfoug@....net> wrote:
> Found it, fixed it. Here are patches for magnum-jumbo, and
> magnum-bleeding.
>
>
>> -----Original Message-----
>> From: Solar Designer [mailto:solar@...nwall.com]
>>
>> magnum, Jim, Dhiru -
>>
>> I don't recall us fixing the bug below, so I guess it's still present?
>>
>> magnum, Jim - can one (or both) of you look into it ASAP and fix it?
>> I think it is very important to have it fixed before 1.7.9-jumbo-6,
>> especially if it's been introduced after -jumbo-5 (I don't know whether
>> this is the case or not).
>>
>> If we don't fix this bug, then at least the SSH format's comment about
>> the use of two different keys in the benchmark is wrong (and needs to be
>> changed). But I'd rather have the bug fixed.
>>
>> On Tue, May 15, 2012 at 12:52:38PM +0400, Solar Designer wrote:
>>> magnum, Jim, Dhiru -
>>>
>>> On Tue, May 15, 2012 at 12:55:36PM +0530, Dhiru Kholia wrote:
>>>> Clean 1.7.9 switches between top two self-test vectors (other
>>>> vectors are tried twice or thrice only). In comparison, jumbo uses
>>>> only top self-test vector ( (other vectors are tried once). Tested
>>>> with MD5 crypt.
>>>
>>> You mean salts, not entire test vectors. It is as expected that clean
>>> 1.7.9 uses just the first two salts when benchmarking. Jumbo's use of
>>> only one salt is a bug. My guess is that it's been introduced along
>>> with support for -1000 and -1001 for BENCHMARK_LENGTH. Can one of you
>>> look into this, please?
>
>
> <FmtBench_one_salt_bug-bleeding.diff>
> <FmtBench_one_salt_bug-mag-jumbo.diff>
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.
