Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20120325021054.GA9207@openwall.com>
Date: Sun, 25 Mar 2012 06:10:54 +0400
From: Solar Designer <solar@...nwall.com>
To: john-dev@...ts.openwall.com
Subject: Re: [JtR patch] Fast cracker for Mozilla Firefox, Thunderbird and SeaMonkey master passwords.

On Sun, Mar 25, 2012 at 04:50:29AM +0400, Solar Designer wrote:
> On Sat, Mar 24, 2012 at 01:12:53PM +0100, magnum wrote:
> > On 03/24/2012 10:25 AM, Dhiru Kholia wrote:
> > > However, I am getting the following warnings during compilation from
> > > the DES code. Can someone take a look and fix them? Thanks!
> > > 
> > > mozilla_des.c: In function ?DES_CBCEn?:
> > > mozilla_des.c:751:2: warning: dereferencing type-punned pointer will
> > > break strict-aliasing rules [-Wstrict-aliasing]
> > 
> > This is more of a follow-up question than an answer :)
> > 
> > The enclosed patch mutes the warnings. Why are the braces needed and
> > what are they "doing"? The format seem to work fine with this patch.
> 
> How did you arrive at the braces?  This is weird stuff.  I am not even
> sure how to parse it (would need to do actual testing to see how gcc
> treats it) - I think this may be either an array initializer or GCC's
> "Statement Exprs" extension (although then you'd probably be required to
> include a semicolon before the closing curly brace, so this is probably
> not it).

Definitely not "Statement Exprs" - these use the ({ ... }) syntax (with
both kinds of braces at once), which is not seen here.

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.