Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Sun, 15 Jul 2012 16:31:08 -0400
From:  <jfoug@....net>
To: john-dev@...ts.openwall.com
Cc: magnum <john.magnum@...hmail.com>
Subject: Re: Another CVS core bug: Does not load bare hashes

---- magnum <john.magnum@...hmail.com> wrote: 
> On 2012-07-15 16:14, jfoug wrote:
> >>> In the bottom of the !ciphertext block:
> >>>
> >>> 		}
> >>> 		*p = 0;
> >>> 		*login = no_username;
> >>> +		fields[1] = *ciphertext;
> >>> 	}
> >>>
> >>> I am testing now, to be 100% sure this does fix 'everything', but I
> >>> believe it will.
> 
> Oh, I thought this was already in your patch. I added it now and indeed
> it fixes the bare hash loading. This should probably be in core as well.

I am 100% sure this should be in core.  core is broken without it, unless you add this logic to every single prepare.

Jim.

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.