janstarke / rexgen

API Documentation
https://github.com/janstarke/rexgen/blob/master/doc/api.md
GNU General Public License v2.0
52 stars 20 forks source link

resume logic corrected for streaming iterator #29

Closed jfoug closed 8 years ago

jfoug commented 8 years ago

This is a true show stopper for JtR. There are still issues that I am digging into, but this allows re-use of the iterator with new word / wordlists. Prior to this change there is always a 'full' run with an empty word, prior to the next word being picked up. And the way JtR works, we have 1 word at a time, so before this correction, this would cause JtR to do twice the amount of work, slowing it down to 1/2 speed (useful speed).

jfoug commented 8 years ago

I have gotten the save/restore fixed fully. This does require a higher version number (the format of the restore string IS changed). I added the state enumeration to each iterator, so that the iterator is actually restored properly. Thus, there is an extra number for each child saved.

jfoug commented 8 years ago

These 2 commits fully fix #28 and #30