Skip to content

Add unified response size checking in wh_CommClient_RecvResponse#453

Open
padelsbach wants to merge 1 commit into
wolfSSL:mainfrom
padelsbach:response-size-checking-common
Open

Add unified response size checking in wh_CommClient_RecvResponse#453
padelsbach wants to merge 1 commit into
wolfSSL:mainfrom
padelsbach:response-size-checking-common

Conversation

@padelsbach

@padelsbach padelsbach commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Alternate approach to #391 with common checking

wolfSSL-Fenrir-bot
wolfSSL-Fenrir-bot reviewed Jul 15, 2026
View reviewed changes

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

Scan targets checked: wolfhsm-core-bugs, wolfhsm-crypto-bugs, wolfhsm-src

Findings: 1
1 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/wh_client.c
@padelsbach padelsbach mentioned this pull request Jul 15, 2026
Closed
@padelsbach
padelsbach force-pushed the response-size-checking-common branch from b8cb516 to d3571fe Compare July 15, 2026 20:10
padelsbach
padelsbach commented Jul 15, 2026
View reviewed changes
Comment thread src/wh_client.c
wolfSSL-Fenrir-bot
wolfSSL-Fenrir-bot reviewed Jul 15, 2026
View reviewed changes

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

⚠️ The automated review timed out after 120 minutes and was terminated. Please retrigger the review manually or contact the Fenrir team.

wolfSSL-Fenrir-bot
wolfSSL-Fenrir-bot reviewed Jul 15, 2026
View reviewed changes

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

Scan targets checked: wolfhsm-core-bugs, wolfhsm-crypto-bugs, wolfhsm-src

Findings: 1
1 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/wh_client.c
@padelsbach
padelsbach force-pushed the response-size-checking-common branch from d3571fe to e3b4344 Compare July 15, 2026 22:59
padelsbach
padelsbach commented Jul 15, 2026
View reviewed changes
Comment thread src/wh_client.c
wolfSSL-Fenrir-bot
wolfSSL-Fenrir-bot reviewed Jul 15, 2026
View reviewed changes

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

Scan targets checked: wolfhsm-core-bugs, wolfhsm-crypto-bugs, wolfhsm-src

No new issues found in the changed files. ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-bot wolfSSL-Fenrir-bot left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@AlexLanzano AlexLanzano

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@padelsbach @wolfSSL-Fenrir-bot @AlexLanzano