Skip to content

20240606-clang-tidy-and-mingw-fixes#7627

Merged
SparkiDev merged 3 commits intowolfSSL:masterfrom
douzzer:20240606-clang-tidy-and-mingw-fixes
Jun 7, 2024
Merged

20240606-clang-tidy-and-mingw-fixes#7627
SparkiDev merged 3 commits intowolfSSL:masterfrom
douzzer:20240606-clang-tidy-and-mingw-fixes

Conversation

@douzzer
Copy link
Copy Markdown
Contributor

@douzzer douzzer commented Jun 6, 2024

fixes for USE_WINDOWS_API:

  • add XWRITE and XREAD definitions to wc_port.h;
  • in wolfSSL_BIO_read(), implement Windows support for XREAD and XWRITE;
  • in wolfSSL_BIO_write_filename(), add 'b' flag to XFOPEN flags;
  • in wolfSSL_RAND_file_name(), add support for XALTHOMEVARNAME, and add Windows definition for it to wc_port.h alongside XWRITE and XREAD.

fixes test_wolfSSL_BIO, test_wolfSSL_X509_print, test_wolfSSL_RAND, test_wolfSSL_RSA_print in cross-mingw-all-crypto scenario.

miscellaneous fixes to clear wolfssl-multi-test.sh ... super-quick-check:

wolfcrypt/src/wc_kyber_poly.c: fix bugprone-macro-parentheses for FROM_MSG_BIT.

wolfcrypt/src/port/riscv/riscv-64-aes.c: fix trailing whitespace.

tested with wolfssl-multi-test.sh ... super-quick-check, which includes cross-mingw-all-crypto, locally fixed to work right, i.e. to actually test --enable-all-crypto.

dgarske
dgarske previously approved these changes Jun 6, 2024
* in wc_port.h, add XWRITE and XREAD definitions and include <io.h>;
* in wolfSSL_BIO_read(), implement Windows support for XREAD and XWRITE;
* in wolfSSL_BIO_write_filename(), add 'b' flag to XFOPEN flags;
* in wolfSSL_RAND_file_name(), add support for XALTHOMEVARNAME, and add Windows definition for it to wc_port.h alongside XWRITE and XREAD.

fixes test_wolfSSL_BIO, test_wolfSSL_X509_print, test_wolfSSL_RAND, test_wolfSSL_RSA_print in cross-mingw-all-crypto scenario.
@douzzer douzzer force-pushed the 20240606-clang-tidy-and-mingw-fixes branch from 714314f to ac5caba Compare June 6, 2024 22:15
@SparkiDev SparkiDev assigned SparkiDev and unassigned dgarske Jun 7, 2024
@SparkiDev SparkiDev merged commit 1c51465 into wolfSSL:master Jun 7, 2024
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.

4 participants