Skip to content

Add secure file I/O utilities, memory wiping, and tests - #278

Open
stenslae wants to merge 4 commits into
wolfSSL:mainfrom
stenslae:pr-secure-io
Open

Add secure file I/O utilities, memory wiping, and tests#278
stenslae wants to merge 4 commits into
wolfSSL:mainfrom
stenslae:pr-secure-io

Conversation

@stenslae

@stenslae stenslae commented Aug 5, 2026

Copy link
Copy Markdown
Member

Added helper functions to remove races in file ops, and refactored sites to call helper functions, and added test coverage. Also fixed CI for no_filesystem builds.

@stenslae stenslae self-assigned this Aug 5, 2026

@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 #278

Scan targets checked: wolfclu-bugs, wolfclu-src

No new issues found in the changed files. ✅

@stenslae
stenslae marked this pull request as ready for review August 5, 2026 22:39
@stenslae stenslae assigned wolfSSL-Bot and stenslae and unassigned stenslae and wolfSSL-Bot Aug 5, 2026
@stenslae
stenslae requested review from wolfSSL-Fenrir-bot and removed request for wolfSSL-Fenrir-bot August 6, 2026 17:05

@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 #278

Scan targets checked: wolfclu-bugs, wolfclu-src

Findings: 9
9 finding(s) posted as inline comments (see file-level comments below)

This review was generated automatically by Fenrir. Findings are non-blocking.

Comment thread src/pkey/clu_rsa.c
Comment thread src/tools/clu_funcs.c
Comment thread src/tools/clu_funcs.c
Comment thread src/tools/clu_funcs.c
Comment thread tests/tools/tools_unit_test.c
Comment thread src/tools/clu_funcs.c
Comment thread src/tools/clu_funcs.c
Comment thread tests/tools/tools_unit_test.c
Comment thread tests/tools/tools_unit_test.c
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.

3 participants