Skip to content

asn: add HAVE_OCSP_RESPONDER guard, to fix wolfboot build.#9959

Merged
JacobBarthelmeh merged 1 commit into
wolfSSL:masterfrom
philljj:fix_wolfboot_build
Mar 12, 2026
Merged

asn: add HAVE_OCSP_RESPONDER guard, to fix wolfboot build.#9959
JacobBarthelmeh merged 1 commit into
wolfSSL:masterfrom
philljj:fix_wolfboot_build

Conversation

@philljj
Copy link
Copy Markdown
Contributor

@philljj philljj commented Mar 12, 2026

Description

Add HAVE_OCSP_RESPONDER define guard around int AsnHashesHash( and friends in asn.c. These functions are only used with the OCSP responder.

Fixes wolfboot build with wolfssl master.

Testing

Build wolfboot sim example with lib/wolfssl set to master.

cp config/examples/sim.config .config
make distclean; make

@philljj philljj self-assigned this Mar 12, 2026
Copilot AI review requested due to automatic review settings March 12, 2026 16:01
@philljj philljj added the For This Release Release version 5.9.2 label Mar 12, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an HAVE_OCSP_RESPONDER compile-time guard around the AsnHashesHash/AsnHashesGetHash helpers in asn.c to fix wolfBoot builds against current wolfSSL master.

Changes:

  • Wrap AsnHashesHash() and AsnHashesGetHash() with #ifdef HAVE_OCSP_RESPONDER / #endif.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@philljj
Copy link
Copy Markdown
Contributor Author

philljj commented Mar 12, 2026

Retest this please.

(Some kind of ocsp-stapling timeout failure in unit test).

@philljj philljj assigned wolfSSL-Bot and unassigned philljj Mar 12, 2026
@JacobBarthelmeh JacobBarthelmeh merged commit c1f71fc into wolfSSL:master Mar 12, 2026
463 checks passed
@philljj philljj deleted the fix_wolfboot_build branch March 15, 2026 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

For This Release Release version 5.9.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants