Skip to content

Zeroize data in private key construction#4562

Merged
gupnik merged 1 commit intodevfrom
gupnik/zeroize
Nov 13, 2025
Merged

Zeroize data in private key construction#4562
gupnik merged 1 commit intodevfrom
gupnik/zeroize

Conversation

@gupnik
Copy link
Copy Markdown
Contributor

@gupnik gupnik commented Nov 13, 2025

Description

This PR zeroizes the bytes post Private key construction.

How to test

Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

  • Create pull request as draft initially, unless its complete.
  • Add tests to cover changes as needed.
  • Update documentation as needed.
  • If there is a related Issue, mention it in the description.

If you're adding a new blockchain

  • I have read the guidelines for adding a new blockchain.

@github-actions
Copy link
Copy Markdown

Binary size comparison

➡️ aarch64-apple-ios: 14.97 MB

➡️ aarch64-apple-ios-sim: 14.97 MB

➡️ aarch64-linux-android: 19.88 MB

➡️ armv7-linux-androideabi: 16.62 MB

➡️ wasm32-unknown-emscripten: 14.31 MB

Copy link
Copy Markdown
Contributor

@satoshiotomakan satoshiotomakan left a comment

Choose a reason for hiding this comment

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

LGTM

@gupnik gupnik merged commit 90d28e6 into dev Nov 13, 2025
15 checks passed
@gupnik gupnik deleted the gupnik/zeroize branch November 13, 2025 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants