Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <028301ce5250$2289f190$679dd4b0$@net>
Date: Thu, 16 May 2013 11:12:16 -0500
From: "jfoug" <jfoug@....net>
To: <john-dev@...ts.openwall.com>
Subject: RE: missing parentheses in dynamic_fmt.c

I am 90% sure this will simply require the full buffer to be swapped if on a
BE system, between the raw md5 and the next call of md5.  It is data written
raw, which would be in LE format (for md5), and thus on the next MD5 that BE
system will swap it.  We need to take that original buffer, put it into BE
format by swapping, so the next run of MD5 will swap it again, and it will
be in proper format.

But yes, this problem with the raw hash is expected (but not wanted), since
this is the first time used, and it was not tested up to this point.

Jim.

From: magnum Sent: Thursday, May 16, 2013 6:52
>On 16 May, 2013, at 13:04 , Jeremi Gosney <epixoip@...dshell.nl> wrote:
>> On 05/16/2013 03:39 AM, magnum wrote:
>>> On 16 May, 2013, at 11:46 , Jeremi Gosney <epixoip@...dshell.nl> wrote:
>>>> dynamic_fmt.c is missing a right parentheses on line 697 that 
>>>> prevents it from compiling on big endian.
>>>> 
>>>> #if !ARCH_LITTLE_ENDIAN
>>>> -    memset(input_buf_X86, 0, FLAT_INP_BUF_SZ;
>>>> +    memset(input_buf_X86, 0, FLAT_INP_BUF_SZ);
>>>> #endif
>>>> 
>>> Committed, thank you. Does it pass a "-t=0 -form=dynamic" on BE after
this fix?
>>> 
>>> magnum
>> 
>> All formats except dynamic_1300 pass their self-tests, which IIRC is a
known issue.
>Thanks. Dynamic_1300 was fixed in 19992ec, 4 days ago but Jim probably
hasn't tried it on BE yet. I'll create an issue on GitHub.
>Oh, I see now it doesn't work with a -generic build on x86 either. So it's
not an endian problem.

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.