Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Tue, 29 Jan 2013 23:42:30 +0100
From: magnum <john.magnum@...hmail.com>
To: john-dev@...ts.openwall.com
Subject: Re: dmg2john

On 29 Jan, 2013, at 23:15 , magnum <john.magnum@...hmail.com> wrote:
> On 29 Jan, 2013, at 22:37 , magnum <john.magnum@...hmail.com> wrote:
> On 29 Jan, 2013, at 2:04 , Solar Designer <solar@...nwall.com> wrote:
>>> On Tue, Jan 29, 2013 at 01:28:57AM +0400, Solar Designer wrote:
>>> I chose to post a different patch in response to Jeremiah's message on
>>> john-users.  That's because there's also a printf format string that
>>> uses "%d", and cno and data_size are also of type int in dmg_fmt_plug.c.
>>> 
>>> The patch that I posted should be good for up to 8 TB.
>> 
> 
> I sent a patch to Jeremiah for trying out. It just adds this (to be used with Solar's patch as well):
> 
> @@ -161,7 +161,7 @@ static void hash_plugin_parse_hash(char *filename)
> 		printf("*%d*", header2.encrypted_keyblob_size);
> 		print_hex(header2.encrypted_keyblob, header2.encrypted_keyblob_size);
> 		printf("*%d*%d*", cno, data_size);
> -		print_hex(chunk + cno * 4096, data_size);
> +		print_hex(chunk + (long)cno * 4096, data_size);
> 		printf("*1*");
> 		print_hex(chunk, 4096);
> 		printf("\n");
> 
> This works for me, except the output is mostly zeros and John can't crack it. Maybe that is the other bug mentioned that I see now?

Unfortunately he got the same mostly zero output with the real file. I give up here.

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.