Skip to content

fixed bug in ifdef logic caused in pull req 6277#6563

Merged
dgarske merged 1 commit intowolfSSL:masterfrom
SKlimaRA:SKlimaRA/fix-funky-ifdef
Jun 30, 2023
Merged

fixed bug in ifdef logic caused in pull req 6277#6563
dgarske merged 1 commit intowolfSSL:masterfrom
SKlimaRA:SKlimaRA/fix-funky-ifdef

Conversation

@SKlimaRA
Copy link
Copy Markdown
Contributor

@SKlimaRA SKlimaRA commented Jun 30, 2023

Description

fixed bug in ifdef logic caused in pull req 6277 around definition of NUM_CUSTOM_EXT

Fixes zd#16385

Testing

How did you test?

Checklist

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

@wolfSSL-Bot
Copy link
Copy Markdown

Can one of the admins verify this patch?

@dgarske
Copy link
Copy Markdown
Member

dgarske commented Jun 30, 2023

OK to test. Contributor agreement on file.

@dgarske dgarske self-assigned this Jun 30, 2023
@dgarske dgarske requested a review from JacobBarthelmeh June 30, 2023 14:45
Copy link
Copy Markdown
Contributor

@JacobBarthelmeh JacobBarthelmeh left a comment

Choose a reason for hiding this comment

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

Reproduced report and confirmed fix with ./configure CPPFLAGS=-DNUM_CUSTOM_EXT=3 --enable-certgen

@dgarske dgarske force-pushed the SKlimaRA/fix-funky-ifdef branch from 4358a21 to c2726b4 Compare June 30, 2023 20:08
@dgarske dgarske merged commit 052fba8 into wolfSSL:master Jun 30, 2023
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.

5 participants