Solar Designer

> I'm seeing this ridiculous inline warning with 560.35.03 / CUDA 12.6.65. I guess you're only seeing it at non-default verbosity? I think we've suppressed it by default (along with...

Hi @cpatulea. Thank you for reporting this. Can you try these: ``` john --format=md5crypt-opencl --test -v=5 john --format=phpass-opencl --test -v=5 john --format=pbkdf2-hmac-md5-opencl --test -v=5 john --format=md5crypt-opencl --skip-self-test shadow ```

Thanks @cpatulea. So the issue is specific to md5crypt-opencl. While we could possibly have a bug in there causing this, that format works just fine on many other devices, including...

I'm no Electrum expert, but seed version 32 looks like it's way _newer_ than what that part of the script supports. Puzzling. Maybe an upgraded wallet (created with older Electrum,...

@skrat Do you intend to provide any further comments here? To me it looks like your original question is not applicable - you ask "why the old versions of Electrum...

@skrat Thanks for confirming this. If you remove these 3 lines from the script: ```python if wallet.get("seed_version") not in (11, 12, 13) and wallet_type != "imported": # all 2.x versions...

@skrat Did you have a chance to try removing those 3 lines? Quite possibly the code readily supports newer wallets and we just didn't know that and thus didn't allow...

Thanks @Layder76. I haven't been keeping track of this issue. It'd help if you clarify what you posted, perhaps separately listing "desired behavior", "actual observed behavior", and "suggested changes". When...

Thanks @Layder76. We also have a number of checks for `xprv` in JtR, including in `electrum2john.py` and `electrum_fmt_plug.c`. Yet you say the "script is ok"? So do we only need...

@Layder76 I've just added the check for `zprv` to `electrum_fmt_plug.c`. Do we need to do anything else? Any change to `electrum2john.py` at all? Perhaps revise the version check? Please suggest...