Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANO7a6yTWG-pnXeBy0Vhq8HN1nHwWLZ-7NQpuzorS3R2Djg70A@mail.gmail.com>
Date: Mon, 1 Apr 2013 14:53:38 +0530
From: Dhiru Kholia <dhiru.kholia@...il.com>
To: john-dev@...ts.openwall.com
Subject: Re: Bug in dynamic (was: new warnings with gcc 4.8.0)

On Mon, Apr 1, 2013 at 2:28 PM, magnum <john.magnum@...hmail.com> wrote:
> On 25 Mar, 2013, at 2:07 , magnum <john.magnum@...hmail.com> wrote:
>> On 24 Mar, 2013, at 9:45 , Dhiru Kholia <dhiru.kholia@...il.com> wrote:
>>> I tried to build unstable-jumbo with gcc 4.8.0 and got some new
>>> warnings.
>>
>>> dynamic_fmt.c: In function ‘init’:
>>> dynamic_fmt.c:842:27: warning: iteration 8u invokes undefined behavior [-Waggressive-loop-optimizations]
>>>    total_len2[i*MMX_COEF] = 32;
>>
>> This one is an actual bug that must be fixed:
>>
>> total_len2 is defined as [BLOCK_LOOPS] but in the loop it's used as if it were MMX_COEF*BLOCK_LOOPS. I'm not sure in what end it's wrong - Jim?
>
> Jim has now fixed this. Please verify latest git with gcc 4.8.0.

Done. dynamic_fmt.c is warning-free now.

-- 
Dhiru

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.