Skip to content

Fixes from Fenrir review#420

Merged
ColtonWilley merged 1 commit into
wolfSSL:masterfrom
SparkiDev:fenrir_fixes_1
Jun 30, 2026
Merged

Fixes from Fenrir review#420
ColtonWilley merged 1 commit into
wolfSSL:masterfrom
SparkiDev:fenrir_fixes_1

Conversation

@SparkiDev

@SparkiDev SparkiDev commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

F-2216, F-2217, F-2218, F-2221
wp_drbg.c

  • wp_drbg_generate, addInLen is now checked and a debug message sent.
  • Added prediction resistance when WP_AHVE_DRBG_RESEED.

wp_pbkdf2.c

  • PBKDF2 check all parameters when required
  • PKCS12 Match behaviour of OpenSSL when deriving and iterations is 0 - do 1 iteration
  • test added for non-FIPS

@SparkiDev SparkiDev self-assigned this Jun 30, 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 #420

Scan targets checked: wolfprovider-bugs, wolfprovider-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/wp_pbkdf2.c Outdated
F-2216, F-2217, F-2218, F-2221
wp_drbg.c
  - wp_drbg_generate, addInLen is now checked and a debug message sent.
  - Added prediction resistance when WP_AHVE_DRBG_RESEED.
wp_pbkdf2.c
  - PBKDF2 check all parameters when required
  - PKCS12 Match behaviour of OpenSSL when deriving and iterations is 0 - do 1 iteration
  - test added for non-FIPS
@ColtonWilley
ColtonWilley merged commit b248975 into wolfSSL:master Jun 30, 2026
72 checks passed
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