Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Mon, 23 Sep 2013 01:23:28 +0200
From: magnum <>
Subject: Re: Daniel's reports #13, #14

On 23 sep 2013, at 00:38, Dániel Bali <> wrote:
> 2013/9/21 Solar Designer <>
>> After a couple of minutes of staring at your code^Whack, I think I see
>> the bug: you set BINARY_SIZE to 2, whereas it should be 4 (or even 8, if
>> you're for some totally weird reason experimenting with 64-bit hash
>> values for now, even though CRC-32 will only use 32-bit).
> BINARY_SIZE started out as 4, when I was testing and didn't modify everything from the keccak format. 
> But I believe in the end the BINARY_SIZE should be 1, because a CRC32 is 32-bit.
> For example raw-md5-opencl has:
> #define CIPHERTEXT_LENGTH   32
> #define DIGEST_SIZE         16
> #define BINARY_SIZE         4

Yes but raw-md5 use reduced binary and then checks the rest in cmp_exact(). The real binary size in raw-md5 is 16. The binary size have to use for CRC32 is 4, for 32 bits.


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.