Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <c9184709d3efbea3c9cabc08062539a7@smtp.hushmail.com>
Date: Wed, 27 Jun 2012 21:22:13 +0200
From: magnum <john.magnum@...hmail.com>
To: john-dev@...ts.openwall.com
Subject: Re: keepass2john.c

On 2012-06-27 15:41, Dhiru Kholia wrote:
> On Wed, Jun 27, 2012 at 10:15 AM, Dhiru Kholia <dhiru.kholia@...il.com> wrote:
>> On Wed, Jun 27, 2012 at 9:32 AM, Solar Designer <solar@...nwall.com> wrote:
>>> For now, though, this is just an unused source file, right?  Maybe I
>>> should exclude it from 1.7.9-jumbo-6?
>>
>> Yes, it should be excluded from jumbo-6 release. I am still in process
>> of developing it. Thanks.
> 
> I have committed KeePass format to magnum-jumbo. So, please include
> keepass2john.c in jumbo-6. Thanks!
> 

I now get this when building linux-x86-64 OMP:

In file included from /usr/include/stdio.h:934:0,
                 from keepass2john.c:25:
/usr/include/x86_64-linux-gnu/bits/stdio2.h: In function ‘process_database’:
keepass2john.c:230:9: warning: ignoring return value of ‘fread’,
declared with attribute warn_unused_result [-Wunused-result]
keepass2john.c: In function ‘keepass2john’:
keepass2john.c:285:11: warning: ‘expectedStartBytesLength’ may be used
uninitialized in this function [-Wuninitialized]
keepass2john.c:191:6: note: ‘expectedStartBytesLength’ was declared here
keepass2john.c:290:6: warning: ‘expectedStartBytes’ may be used
uninitialized in this function [-Wuninitialized]
keepass2john.c:190:17: note: ‘expectedStartBytes’ was declared here
keepass2john.c:283:11: warning: ‘initializationVectorsLength’ may be
used uninitialized in this function [-Wuninitialized]
keepass2john.c:189:6: note: ‘initializationVectorsLength’ was declared here
keepass2john.c:289:6: warning: ‘initializationVectors’ may be used
uninitialized in this function [-Wuninitialized]
keepass2john.c:188:17: note: ‘initializationVectors’ was declared here
keepass2john.c:281:11: warning: ‘transformSeedLength’ may be used
uninitialized in this function [-Wuninitialized]
keepass2john.c:187:6: note: ‘transformSeedLength’ was declared here
keepass2john.c:288:6: warning: ‘transformSeed’ may be used uninitialized
in this function [-Wuninitialized]
keepass2john.c:186:17: note: ‘transformSeed’ was declared here
keepass2john.c:279:11: warning: ‘masterSeedLength’ may be used
uninitialized in this function [-Wuninitialized]
keepass2john.c:185:6: note: ‘masterSeedLength’ was declared here
keepass2john.c:287:6: warning: ‘masterSeed’ may be used uninitialized in
this function [-Wuninitialized]
keepass2john.c:184:17: note: ‘masterSeed’ was declared here

I would really hate to have this in the release.

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.