Skip to content

Fixes for building the compatibility layer with no realloc#7695

Merged
JacobBarthelmeh merged 2 commits intowolfSSL:masterfrom
dgarske:compat_realloc
Jul 1, 2024
Merged

Fixes for building the compatibility layer with no realloc#7695
JacobBarthelmeh merged 2 commits intowolfSSL:masterfrom
dgarske:compat_realloc

Conversation

@dgarske
Copy link
Copy Markdown
Member

@dgarske dgarske commented Jun 27, 2024

Description

  • Fixes for building the compatibility layer with WOLFSSL_NO_REALLOC.

  • Improve benchmark FreeRTOS default tick rate logic. For example Xilinx FreeRTOS uses 10ms tick (not default 1ms), so include configTICK_RATE_HZ in calculation if available.

  • Fix test.c warning around too many parens with no realloc.

ZD 18224

Testing

./configure --enable-opensslall CFLAGS="-DWOLFSSL_NO_REALLOC" && make check

Checklist

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

… Tested using `./configure --enable-opensslextra CFLAGS="-DWOLFSSL_NO_REALLOC"`.

Improve benchmark FreeRTOS default tick rate logic. For example Xilinx FreeRTOS uses 10ms tick (not default 1ms), so include `configTICK_RATE_HZ` in calculation if available.
Fix test.c warning around too many parens with no realloc.
@dgarske dgarske added the For This Release Release version 5.9.1 label Jun 27, 2024
@dgarske dgarske requested a review from SparkiDev June 27, 2024 23:04
@dgarske
Copy link
Copy Markdown
Member Author

dgarske commented Jun 28, 2024

Retest this please

SparkiDev
SparkiDev previously approved these changes Jun 28, 2024
@dgarske dgarske removed their assignment Jun 28, 2024
@dgarske dgarske requested a review from JacobBarthelmeh June 28, 2024 13:02
@JacobBarthelmeh JacobBarthelmeh merged commit 3206637 into wolfSSL:master Jul 1, 2024
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.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants