[PATCH] cracknames: add python test & fix issues (bug #12842)
Andrew Bartlett
abartlet at samba.org
Mon Jul 10 05:18:58 UTC 2017
On Mon, 2017-07-10 at 13:31 +1200, Bob Campbell via samba-technical
wrote:
> Hi again,
>
> Turns out that I actually broke the existing cracknames test with this;
> oops. Attached is a patch which passes. The change is using a different
> error code when it can't find the expected format.
>
> Thanks,
> Bob
Thanks Bob. Can you confirm you have tested both against Windows?
One nit, normally the BUG lines go with the signed-off-by etc, not
first in the commit message. Other than that:
Reviewed-by: Andrew Bartlett <abartlet at samba.org>
Thanks!
Andrew Bartlett
--
Andrew Bartlett http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
Samba Developer, Catalyst IT http://catalyst.net.nz/services/samba
More information about the samba-technical
mailing list