Skip to content

Fixes for ECC import key zeroization, ant test native library paths - #302

Open
cconlon wants to merge 2 commits into
wolfSSL:masterfrom
cconlon:fenrirOct16_3
Open

cconlon wants to merge 2 commits into
wolfSSL:masterfrom
cconlon:fenrirOct16_3

Conversation

@cconlon

@cconlon cconlon commented Oct 16, 2026

Copy link
Copy Markdown
Member

This PR fixes 2 Fenrir issues:

  • F-15084: Zeroize the JNI private key copy on the GetStringUTFChars() failure path in ECC import.
  • F-15085: Drop the literal $JAVA_HOME and unset Windows DLL dirs from the ant test native library paths.

@cconlon cconlon self-assigned this Oct 16, 2026
@cconlon
cconlon requested review from wolfSSL-Fenrir-bot and a balanced review from Copilot October 16, 2026 18:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fenrir Automated Review — PR #302

Scan targets checked: wolfcrypt-jni-src, wolfcrypt-jni-bugs
Not reviewed (no scan target covers them): build.xml
Coverage: 1 of 1 in-scope changed file(s) opened by the reviewer

Fenrir result: Approved ✅

No new issues found in the changed files.

Advisory only — this automated result does not count as a GitHub approval.

Review tier: Lite

@cconlon cconlon assigned rlm2002 and unassigned cconlon Oct 16, 2026
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.

4 participants