Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4E519635.7050807@bredband.net>
Date: Mon, 22 Aug 2011 01:35:17 +0200
From: magnum <rawsmooth@...dband.net>
To: john-dev@...ts.openwall.com
Subject: Re: Update to wordlist, (and some portability issues)

On 2011-08-22 01:01, JFoug wrote:
 > Are you sure you are running the most current version?

Lol, I just identified one problem: you accidentally removed pkzip from 
the wiki when you added 0026, so I never saw rev 02 of it!

After digging a little in the wiki history, I have now applied it.

It now DOES pass self-test built with gcc, and --inc cracks the defcon 
contest zip files in minutes.

However, valgrind is not 100% happy:

magnum@...ge:~/src/john/john/src$ valgrind --track-origins=yes 
../run/john -test -fo:pkzip
==9574== Memcheck, a memory error detector
==9574== Copyright (C) 2002-2010, and GNU GPL'd, by Julian Seward et al.
==9574== Using Valgrind-3.6.1 and LibVEX; rerun with -h for copyright info
==9574== Command: ../run/john -test -fo:pkzip
==9574==
Benchmarking: pkzip [N/A]...
==9574== Conditional jump or move depends on uninitialised value(s)
==9574==    at 0x56A0510: inflateReset2 (in 
/lib/x86_64-linux-gnu/libz.so.1.2.3.4)
==9574==    by 0x56A0605: inflateInit2_ (in 
/lib/x86_64-linux-gnu/libz.so.1.2.3.4)
==9574==    by 0x45D987: cmp_exact (pkzip_fmt_plug.c:714)
==9574==    by 0x469E5C: fmt_self_test (formats.c:96)
==9574==    by 0x463876: benchmark_format (bench.c:136)
==9574==    by 0x463EA1: benchmark_all (bench.c:397)
==9574==    by 0x46CE7C: main (john.c:464)
==9574==  Uninitialised value was created by a heap allocation
==9574==    at 0x4C28FAC: malloc (vg_replace_malloc.c:236)
==9574==    by 0x56A05E1: inflateInit2_ (in 
/lib/x86_64-linux-gnu/libz.so.1.2.3.4)
==9574==    by 0x45D987: cmp_exact (pkzip_fmt_plug.c:714)
==9574==    by 0x469E5C: fmt_self_test (formats.c:96)
==9574==    by 0x463876: benchmark_format (bench.c:136)
==9574==    by 0x463EA1: benchmark_all (bench.c:397)
==9574==    by 0x46CE7C: main (john.c:464)
==9574==
==9574== Conditional jump or move depends on uninitialised value(s)
==9574==    at 0x56A0510: inflateReset2 (in 
/lib/x86_64-linux-gnu/libz.so.1.2.3.4)
==9574==    by 0x56A0605: inflateInit2_ (in 
/lib/x86_64-linux-gnu/libz.so.1.2.3.4)
==9574==    by 0x45D6C0: cmp_exact (pkzip_fmt_plug.c:641)
==9574==    by 0x469E5C: fmt_self_test (formats.c:96)
==9574==    by 0x463876: benchmark_format (bench.c:136)
==9574==    by 0x463EA1: benchmark_all (bench.c:397)
==9574==    by 0x46CE7C: main (john.c:464)
==9574==  Uninitialised value was created by a heap allocation
==9574==    at 0x4C28FAC: malloc (vg_replace_malloc.c:236)
==9574==    by 0x56A05E1: inflateInit2_ (in 
/lib/x86_64-linux-gnu/libz.so.1.2.3.4)
==9574==    by 0x45D6C0: cmp_exact (pkzip_fmt_plug.c:641)
==9574==    by 0x469E5C: fmt_self_test (formats.c:96)
==9574==    by 0x463876: benchmark_format (bench.c:136)
==9574==    by 0x463EA1: benchmark_all (bench.c:397)
==9574==    by 0x46CE7C: main (john.c:464)
==9574==
DONE
Raw:	458265 c/s real, 467523 c/s virtual


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.