Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date: Fri, 13 Jul 2012 10:51:48 +0400
From: Solar Designer <>
Subject: prepare()

Jim, all -

Attached is a patch that shows how I am introducing prepare() into core
tree.  While working on this, I noticed a few issues with this stuff in
jumbo.  Here are two that I can recall:

1. Parsing of /etc/master.passwd format files is broken in jumbo.  1.7.9
release (and older releases) would extract GECOS, home directory, and
shell out of /etc/master.passwd format files in typical cases, whereas
jumbo would not.  (My patch preserves and enhances this functionality.)

I understand that the way this was implemented was hackish and unclear
(there was no comment), so no wonder it got dropped when someone else
reworked the code.  (I've made it cleaner and added a comment now.)

2. The first alt format loop in ldr_split_line() runs with the
ciphertext prepare()d for the currently detected format, but then passes
this ciphertext into alt formats' valid().  I think it should use those
formats' prepare() for that.  (My patch does.)


View attachment "john-" of type "text/plain" (21952 bytes)

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.