Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Fri, 22 Jul 2011 14:18:04 +0200
From: magnum <>
Subject: Re: ISO-8859-1 casing (experimental)

On 2011-07-19 23:30, JimF wrote:
>> Am 18.07.2011 23:39, schrieb jfoug:
>>> I created a new option (--iso-8859-1). This is just a boolean flag
>>> (like the --utf8 switch).
>> At this point, shouldn't this be converted into an enum,
>> allowing future enhancements?
>> Frank
> This is a VERY good suggestion. There is a very limited set of flag
> bits, but we may want to expand this to additional charsets at some
> time. That is why this is only a 'john-dev' posting, as I was hoping for
> others input such as this.
> We may still want a flag bit, telling john that there 'is' a charset
> change. That way, we have a simply quick flag check to know that we are
> in default '7-bit' mode, and do not slow anything down. However, if the
> flag is set, we can take the additional step to see what charset we need
> (when we are in the proper section of code).

I'm checking this patch out now.

+       {"iso-8859-1", FLG_ISO5998_1, FLG_ISO5998_1},//, 

Why is it called ISO5998_1?

+       if (options.flags & FLG_ISO5998_1) {
+               conv_tolower = rules_init_conv(CHARS_UPPER 
+               conv_toupper = rules_init_conv(CHARS_LOWER 
+       } else {
+               conv_tolower = rules_init_conv(CHARS_UPPER, CHARS_LOWER);
+               conv_toupper = rules_init_conv(CHARS_LOWER, CHARS_UPPER);
+       }

If using an enum instead, things like the above would be a switch 
statement, right?

The whole thing looks pleasantly simple. I like it! It could be improved 
to affect all uc/lc stuff within formats (I think LM, NETLM, NETHALFLM 
and sapB are affected). These could use a shared --charset aware uc (or 
lc) function. Once they do, adding a codepage to that shared function 
would affect all applicable formats. But we'd have to beware of 
performance drops.


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.