[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 29 Mar 2012 08:26:31 +0200
From: magnum <john.magnum@...hmail.com>
To: john-dev@...ts.openwall.com
Subject: Re: mozilla format minor problems
On 03/29/2012 06:52 AM, Dhiru Kholia wrote:
> On Thu, Mar 29, 2012 at 3:01 AM, magnum <john.magnum@...hmail.com> wrote:
>> Dhiru,
>>
>> I had the strangest segfault problems in latest committed mozilla
>> format, that ended up being because I had the old format infiles - I
>> just needed to re-run mozilla2john (I knew that but I forgot it!). You
>> may want to handle invalid input files better (perhaps just make valid()
>> less promiscuos).
>
> I will try to validate the complete ciphertext in valid() by calling
> strtok in a loop and checking the times it returned valid strings.
> What do you think about this idea?
Sure. Just a few trivial checks of corrects minimum or absolute lengths.
Actually almost no-one will have old mozilla2john format files as it was
not in last jumbo :)
>> Another issue: When building without NSS installed (and HAVE_NSS
>> disabled of course), I get the following output when building:
>>
>> Package nss was not found in the pkg-config search path.
>> Perhaps you should add the directory containing `nss.pc'
>> to the PKG_CONFIG_PATH environment variable
>> No package 'nss' found
>> Package nss was not found in the pkg-config search path.
>> Perhaps you should add the directory containing `nss.pc'
>> to the PKG_CONFIG_PATH environment variable
>> No package 'nss' found
>>
>> I think we should try to mute them though I'm not sure how. Just adding
>> --silence-errors to pkg-config would not handle the case where a system
>> doesn't even have pkg-config installed.
>
> Currently we have.
>
> CFLAGS = -c -Wall -O2 .. `pkg-config --cflags nss`
>
> Instead of this can we have:
>
> ifdef HAVE_NSS
> CFLAGS = -c -Wall -O2 ... `pkg-config --cflags nss`
> else
> CFLAGS = -c -Wall -O2 ... $(JOHN_CFLAGS)
>
> And sample applies to LDFLAGS. Do you think this will solve the problem?
This will be confusing (duplicated stuff). Maybe something like this is
better, I did not test but I think it would work:
# NSS (and NSPR) flag, un-comment this line if installed (enables
mozilla format)
#HAVE_NSS = -DHAVE_NSS `pkg-config --cflags nss`
CFLAGS = -DRAR_USE_EVP -c -Wall -O2 -fomit-frame-pointer
-I/usr/local/include $(HAVE_NSS) $(OMPFLAGS) $(JOHN_CFLAGS)
magnum
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.
