Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20150504153238.GA19203@openwall.com>
Date: Mon, 4 May 2015 18:32:39 +0300
From: Solar Designer <solar@...nwall.com>
To: john-dev@...ts.openwall.com
Subject: Re: External problem

Hi Kai,

On Mon, May 04, 2015 at 08:38:10PM +0800, Kai Zhao wrote:
> I found a bug using asan when "--external=LanMan". But magnum
> and I are not sure whether it is really a bug. So could you help me
> to solve this problem?
> 
> The problem is described here:
> 
> https://github.com/magnumripper/JohnTheRipper/issues/1149

Why, per magnum's comments, he does confirm this is indeed a bug.  It looks
like there's been an understanding problem between the two of you.

Since this is core John code, you should have brought the problem to
john-dev to my attention much sooner.

> I think it is really a bug, since there is the case that it tries to visit
> (sp-2)  when sp points to c_stack[0].

Yes.  I've just fixed it:

"In GCC-specific c_execute_fast(), initially set sp to &c_stack[2] so that
there's room for op_push_* to spill imm to stack even when there wasn't
actually a previous top of stack value to cache (since we're at the top level).
It is simpler and quicker to let them do it than to treat this as a special
case in the code.

Bug found and reported by Kai Zhao:
http://www.openwall.com/lists/john-dev/2015/05/04/8 "

http://cvsweb.openwall.com/cgi/cvsweb.cgi/Owl/packages/john/john/src/compiler.c.diff?r1=1.15;r2=1.16

Thank you!

Alexander

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.