Skip to content

Fix for building LMS with verify only #8913

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 23, 2025
Merged

Conversation

dgarske
Copy link
Contributor

@dgarske dgarske commented Jun 23, 2025

Description

Fix for building LMS with verify only. Added tests for LMS/XMSS verify only. New wc_LmsKey_GetKid references key->priv_raw that is not available.

Broken in PR #8786

Testing

See wolfBoot test failures from wolfSSL/wolfBoot#576

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

…y only. New `wc_LmsKey_GetKid` references `key->priv_raw` that is not available.
@dgarske dgarske self-assigned this Jun 23, 2025
dgarske added a commit to dgarske/wolfBoot that referenced this pull request Jun 23, 2025
@dgarske
Copy link
Contributor Author

dgarske commented Jun 23, 2025

Jenkins retest this please:

  • Found unhandled hudson.remoting.RequestAbortedException exception:
  • Found unhandled org.jenkinsci.plugins.workflow.support.steps.AgentOfflineException exception:

@dgarske dgarske assigned anhu and wolfSSL-Bot and unassigned dgarske Jun 23, 2025
@dgarske dgarske requested a review from anhu June 23, 2025 20:10
@anhu anhu merged commit aec1392 into wolfSSL:master Jun 23, 2025
290 of 291 checks passed
dgarske added a commit to dgarske/wolfBoot that referenced this pull request Jun 23, 2025
danielinux pushed a commit to wolfSSL/wolfBoot that referenced this pull request Jun 24, 2025
@dgarske dgarske deleted the pq_verifyonly branch June 27, 2025 19:44
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