Skip to content

fix(offline-keys): use CryptoCtx for Iguana get_private_keys#2683

Merged
shamardy merged 1 commit intodevfrom
hotfix/iguana-offline-key
Nov 10, 2025
Merged

fix(offline-keys): use CryptoCtx for Iguana get_private_keys#2683
shamardy merged 1 commit intodevfrom
hotfix/iguana-offline-key

Conversation

@shamardy
Copy link
Copy Markdown
Collaborator

Fixes a critical bug where get_private_keys in iguana mode derived keys from an empty passphrase when ctx.conf["passphrase"] is null (when loading it from KDF encrypted storage) instead of the active wallet secret, returning incorrect keys.

Fixes #2682

@shamardy shamardy added priority: urgent Critical tasks requiring immediate action. 2.6.0-beta labels Nov 10, 2025
@smk762 smk762 self-requested a review November 10, 2025 13:51
Copy link
Copy Markdown

@smk762 smk762 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested this in Komodo Wallet dev branch, using flutter sdk from https://github.com/KomodoPlatform/komodo-defi-sdk-flutter/tree/fix_priv_key_export and was returned expected keys, matching address/pk output in legacy desktop.

image

Pubkey for same mnemonic via postman resolves to expected address.

Note: Existing known iguana segwit pk handling bug still present.

@shamardy shamardy merged commit 1874d43 into dev Nov 10, 2025
23 of 30 checks passed
@shamardy shamardy deleted the hotfix/iguana-offline-key branch November 10, 2025 14:40
DeckerSU added a commit to GLEECBTC/komodo-defi-sdk-flutter that referenced this pull request Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.6.0-beta priority: urgent Critical tasks requiring immediate action.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants