Skip to content

CryptoPkg: Add SHA-256/384/512 PBKDF2 unit tests - #1863

Draft
Flickdm wants to merge 1 commit into
microsoft:release/202511from
Flickdm:feat/pkcs5-pbkdf2-sha384-sha512
Draft

CryptoPkg: Add SHA-256/384/512 PBKDF2 unit tests#1863
Flickdm wants to merge 1 commit into
microsoft:release/202511from
Flickdm:feat/pkcs5-pbkdf2-sha384-sha512

Conversation

@Flickdm

@Flickdm Flickdm commented Jul 24, 2026

Copy link
Copy Markdown
Member

Description

Extend the PKCS#5 PBKDF2 host unit test to cover additional digest sizes. The test previously validated only HMAC-SHA1; refactor it to be data driven and add known-answer vectors for HMAC-SHA256, HMAC-SHA384 and HMAC-SHA512 derived keys, exercising the newly supported digest sizes in Pkcs5HashPassword.

For details on how to complete these options and their meaning refer to CONTRIBUTING.md.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

N/A

Integration Instructions

N/A

@mu-automation

mu-automation Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

✅ QEMU Validation Passed

Source Dependencies

Repository Commit
mu_basecore 0cb2012
mu_tiano_platforms d8ecfb8

Results

Platform Target Build Boot Overall Boot Time Build Logs Boot Logs
Q35 DEBUG ✅ success ✅ success 0m 18s Build Logs Boot Logs
ArmVirt DEBUG ✅ success ✅ success 0m 14s Build Logs Boot Logs

Workflow run: https://github.com/microsoft/mu_basecore/actions/runs/30120126391

This comment was automatically generated by the Mu QEMU PR Validation workflow.

Extend the PKCS#5 PBKDF2 host unit test to cover additional digest
sizes. The test previously validated only HMAC-SHA1; refactor it to
be data driven and add known-answer vectors for HMAC-SHA256,
HMAC-SHA384 and HMAC-SHA512 derived keys, exercising the newly
supported digest sizes in Pkcs5HashPassword.

Signed-off-by: Doug Flick <dougflick@microsoft.com>
@Flickdm
Flickdm force-pushed the feat/pkcs5-pbkdf2-sha384-sha512 branch from 96a9698 to 0cb2012 Compare July 24, 2026 19:09
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.

1 participant