Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 17 Jan 2013 13:22:01 +0100
From: magnum <>
Subject: NSS not needed (was: old clang warning in mozilla_des.c)

On 16 Jan, 2013, at 4:14 , Dhiru Kholia <> wrote:
> On Jan 16, 2013 4:20 AM, "Frank Dittrich" <> wrote:
> >
> > clang correctly points out that inLen < 0 is alway false in
> > mozilla_des.c line 833, because inLen is unsigned int.
> >
> > When I searched for users of DES_Encrypt(), I found none.
> >
> > Removing DES_Encrypt() from mozilla_des.c and mozilla_des.h still allows
> > building john, with or without -DHAVE_NSS.
> Yes,  the format can be built without NSS.
> I suspected this earlier and now you have confirmed it.
> I will take a look at it soon. Thanks Frank. 

This would be nice for Jumbo-8, please commit to unstable if you fix it. Remove all references to HAVE_NSS in Makefile too.

Content of type "text/html" skipped

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.