Date: Sun, 10 Jun 2012 11:10:18 +0200 From: Frank Dittrich <frank_dittrich@...mail.com> To: john-dev@...ts.openwall.com Subject: Re: Patch: allow --markov=SECTION in addition to --markov[=[MIN_LEVEL-]LEVEL[:START[:END[:[MIN_LENGHT-]LENGTH]]]] On 06/10/2012 09:49 AM, Simon Marechal wrote: > On 06/09/2012 09:18 PM, Frank Dittrich wrote: >> I did not see any negative side effects, but please feel free to test >> the patch and/or suggest changes. > > I should really read the code instead of asking, No problem. (You could also have looked at the doc/MARKOV changes in the patch.) > but I am not sure I > will have time for this soon, so here it is : is it still possible to > run like this : > > -markov:300:0:1148673650879:24 -sess:markov0 > -markov:300:1148673650879:2297347301758:24 -sess:markov1 > -markov:300:2297347301758:3446020952637:24 -sess:markov2 > -markov:300:3446020952637:4594694603516:24 -sess:markov3 > -markov:300:4594694603516:5743368254395:24 -sess:markov4 > -markov:300:5743368254395:0:24 -sess:markov5 This is still possible. When the first character after -markov: is not a letter, the old behavior is still in place. > > (this is the 6 cores run for maxlen=24 and level=300) > > This would probably look like that : > > -markov:confxxx -markov:300:0:1148673650879:24 -sess:markov0 No, using two --markov options is not supported. Currently, you have to use either the old logic, or the new one. But it could be extended to support the combination of both versions --markov:section:<more_parameters> or --markov=section:<more_parameters> Even if we commit the current patch as it is right now, the next step would just be another enhancement, not a change that is not compatible with any previous logic. (I'll have to check what currently happens with --markov=a:100 and a section [Markov:a:100] in john.conf. May be I have to add a check that the section name doesn't contain a colon. Currently this requirement is only mentioned in doc/MARKOV.) So, your example could look like -markov:confxxx:300:0:1148673650879:24 -sess:markov0 -markov:confxxx:300:1148673650879:2297347301758:24 -sess:markov1 -markov:confxxx:300:2297347301758:3446020952637:24 -sess:markov2 -markov:confxxx:300:3446020952637:4594694603516:24 -sess:markov3 -markov:confxxx:300:4594694603516:5743368254395:24 -sess:markov4 -markov:confxxx:300:5743368254395:0:24 -sess:markov5 or --markov=confxxx:300:0:1148673650879:24 -sess:markov0 --markov=confxxx:300:1148673650879:2297347301758:24 -sess:markov1 --markov=confxxx:300:2297347301758:3446020952637:24 -sess:markov2 --markov=confxxx:300:3446020952637:4594694603516:24 -sess:markov3 --markov=confxxx:300:4594694603516:5743368254395:24 -sess:markov4 --markov=confxxx:300:5743368254395:0:24 -sess:markov5 And your john.conf must contain a section [Markov:confxxx] or [Markov:ConfXXX] since the names are not case sensitive. If we extend the logic this way, the distinction between the --markov[=[MIN_LEVEL-]LEVEL[:START[:END[:[MIN_LENGHT-]LENGTH]]]] logic and the --markov=SECTION[:[MIN_LEVEL-]LEVEL[:START[:END[ [MIN_LENGHT-]LENGTH]]]]] logic would be that SESSION begins with a letter (or contains at least one letter). For the --markov:SECTION:<more_parameters> case, I suggest that parameters specified on the command line overwrite parameters specified in the section [Markov:SECTION] So with this section: [Markov:example] Statsfile = $JOHN/my_stats MkvLevel = 250 MkvMaxLen = 10 and a command line parameter --markov=example:180 this should be the same as --markov:180:0:0:10 except that the stats file is $JOHN/my_stats instead of $JOHN/stats. (Assuming that [Options:Markov] contains Statsfile = $JOHN/stats.) For your above example, all possible parameters are specified on the command line, so that just the Statsfile is read from [Markov:example] instead of [Options:Markov]. This should be fairly easy to implement. This would even allow to define a Markov section with just one line: [Markov:special_stats] Statsfile = $JOHN/special_stats Currently I require at least MkvMaxLen and MkvLvl, but if you specify those on the command line, there is no need to add them to the Markov section. The only difficulty would be the "user interface design" decision how to take care of missing MIN_LEVEL and/or MIN_LENGTH parameters on the command line if the section [Markov:Example] contains these lines: MkvMinLvl = 220 MkvMinLength = 8 In this case, I'd suggest that --markov=example:300:0:1148673650879:24 would be the same as --markov=example:220-300:0:1148673650879:8-24 That means, if MIN_LEVEL is not specified, it will be read from the variable MkvMinLvl in section [Markov:Example]. To avoid this, you'd have to use --markov=example:0-300:0:1148673650879:24 Similarly, to avoid MIN_LENGTH=8, you'd have to use --markov=example:300:0:1148673650879:0-24 How does that sound? Should we go this route? Frank
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.