Skip to content

Commit

Permalink
mbedTLS: Fix build without the module on MinGW
Browse files Browse the repository at this point in the history
The `include/mbedtls/check_config.h` header file has a generic check
which requires `MBEDTLS_PLATFORM_C` to be defined on Windows when
building with MinGW and ancient MSVC versions.

The few mbedTLS files we include in builds without the module don't
really require anything from the mbedTLS platform file beside the
`mbedtls_platform_zeroize` function which we already override with a
custom one by defining `MBEDTLS_PLATFORM_ZEROIZE_ALT`.

So we simply add the same check and force the define in such case.
  • Loading branch information
Faless committed Apr 15, 2024
1 parent c951421 commit 060097b
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions thirdparty/mbedtls/include/godot_core_mbedtls_config.h
Original file line number Diff line number Diff line change
Expand Up @@ -50,4 +50,11 @@
#define MBEDTLS_PLATFORM_ZEROIZE_ALT
#define MBEDTLS_NO_DEFAULT_ENTROPY_SOURCES

// This is only to pass a check in the mbedtls check_config.h header, none of
// the files we include as part of the core build uses it anyway, we already
// define MBEDTLS_PLATFORM_ZEROIZE_ALT which is the only relevant function.
#if defined(__MINGW32__)
#define MBEDTLS_PLATFORM_C
#endif

#endif // GODOT_CORE_MBEDTLS_CONFIG_H

0 comments on commit 060097b

Please sign in to comment.