Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 14 Jun 2012 14:53:28 +0200
From: magnum <john.magnum@...hmail.com>
To: john-dev@...ts.openwall.com
Subject: Re: Compiler warnings

On 2012-06-13 22:49, magnum wrote:
> On 2012-06-13 22:32, Lukas Odzioba wrote:
>> What you've sent me was complete list? I assume that they are from
>> 64bit build.
>> I fixed some..: that's what's left:
>> 32bit
>> http://ideone.com/RMDPT
>> http://ideone.com/CuIP0
>
> Thanks, yes it was a complete list from gcc 4.6.3. Some of these were
> new to me, some others (NT and mscash1) I too saw today when using gcc
> 4.4. These warnings tend to change with compiler version. I'll have a
> look at RAR and the others.
>
> Jim, the NT and mscash1 warnings shown in the links above are from your
> Sparc fixes. The weird things are:
>
> 1) I just can not understand how they break aliasing rules (we cast to
> shorter)
>
> 2) In NS_fmt_plug.c, *exactly* the same type of code is used, yet it
> does not produce warnings. Why?!

I refreshed my understanding about aliasing, and fixed the two formats 
using a union. Now there is no warnings at all on my gear (even with gcc 
4.4), except disabled formats that are supposed to make notices. I still 
can't build 32-bit though.

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.