[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <55FAA0A3.8000100@mailbox.org>
Date: Thu, 17 Sep 2015 13:14:43 +0200
From: Frank Dittrich <frank.dittrich@...lbox.org>
To: john-dev@...ts.openwall.com
Subject: Re: multi-threaded hash table initialization
On 09/17/2015 12:53 PM, Frank Dittrich wrote:
> On 09/17/2015 12:37 AM, magnum wrote:
>> FYI your patch doesn't apply cleanly to current bleeding (despite, I'm
>> pretty sure, there should be no clash at all with recent commits). This
>> happened with one of your patches yesterday also and it's probably very
>> trival this time too but it'll take me some head scratching and thus
>> some delay. It's not really a problem, just wanted to let you know.
>
> It also causes a warning for non-OMP builds:
>
> loader.c: In function ‘ldr_fix_database’:
> loader.c:1508:2: warning: ‘hash_alloc’ may be used uninitialized in this
> function [-Wmaybe-uninitialized]
> memset(salt->hash, 0, hash_alloc);
> ^
> loader.c:1409:23: note: ‘hash_alloc’ was declared here
> size_t bitmap_alloc, hash_alloc
clang 3.5.0 (no openmp support) has these warnings:
/usr/bin/ar: creating aes.a
loader.c:211:18: warning: comparison of unsigned expression >= 0 is
always true [-Wtautological-compare]
} while (--size >= 0);
~~~~~~ ^ ~
loader.c:212:11: warning: comparison of unsigned expression < 0 is
always false [-Wtautological-compare]
if (size < 0)
~~~~ ^ ~
loader.c:1434:6: warning: variable 'hash_alloc' is used uninitialized
whenever 'if' condition is false [-Wsometimes-uninitialized]
if (hash_size > 1) {
^~~~~~~~~~~~~
loader.c:1508:24: note: uninitialized use occurs here
memset(salt->hash, 0, hash_alloc);
^~~~~~~~~~
loader.c:1434:2: note: remove the 'if' if its condition is always true
if (hash_size > 1) {
^~~~~~~~~~~~~~~~~~~
loader.c:1409:33: note: initialize the variable 'hash_alloc' to silence
this warning
size_t bitmap_alloc, hash_alloc;
^
= 0
3 warnings generated.
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.
