Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 30 May 2013 01:01:07 +0200
From: magnum <>
Subject: Re: Unify internal form of mscash2 hashes?

On 28 May, 2013, at 10:18 , Sayantan Datta <> wrote:
> Hi ,
> On Tue, May 28, 2013 at 12:51 PM, magnum <> wrote:
> Sayantan, will you have a look at this please? I think you merely need to take prepare() as-is from the CPU format.
> Cool, no problem. 

Yet you wrote your own again? I suggest you use the well tested valid(), split() and prepare() from the CPU format. There is really no need to invent worse wheels. I can crash your latest version like this:

$ echo aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa > test
$ ../run/john -form:mscash2-opencl test
Segmentation fault: 11

If I replace those three functions with the CPU versions, problem goes away.


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.