Repository navigation
sanity checks on truncation of input lengths - #11945
JacobBarthelmeh wants to merge 5 commits into
Conversation
|
wolfSSL-Fenrir-bot
left a comment
There was a problem hiding this comment.
Fenrir Automated Review — PR #11945
Scan targets checked: wolfssl-src, wolfssl-bugs
Coverage: 3 of 4 in-scope changed file(s) opened by the reviewer; not opened: tests/api/test_ossl_mac.c
Findings: 1
1 finding(s) posted as inline comments (see file-level comments below)
This review was generated automatically by Fenrir. Reported findings require changes before merge.
Review tier: Lite
1b2119b to
70febf6
Compare
wolfSSL-Fenrir-bot
left a comment
There was a problem hiding this comment.
Fenrir Automated Review — PR #11945
Scan targets checked: wolfssl-src, wolfssl-bugs
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
Fenrir's latest completed scan found no issues; clearing the prior automated change request.
|
Retest this please Jenkins |
|
|
||
| /* wolfCrypt takes a word32 length - fail rather than truncate. */ | ||
| if (!WOLFSSL_CRYPTO_LEN_FITS_WORD32(sz)) { | ||
| return 0; |
There was a problem hiding this comment.
Add WOLFSSL_MSG calls to all of these.
F-16878