From 913fc5fff3831057f03b1de529c76ecd65e34b26 Mon Sep 17 00:00:00 2001 From: Gilles Peskine Date: Thu, 23 Sep 2021 17:43:46 +0200 Subject: [PATCH] Better default for MBEDTLS_CHECK_RETURN in config.h An empty expansion is possible, but as documented its effect is to disable the feature, so that isn't a good example. Instead, use the GCC implementation as the default: it's plausible that it could work even on compilers that don't advertise themselves as sufficiently GCC-like to define __GNUC__, and if not it gives users a concrete idea of what the macro is supposed to do. Signed-off-by: Gilles Peskine --- include/mbedtls/mbedtls_config.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/mbedtls/mbedtls_config.h b/include/mbedtls/mbedtls_config.h index 2110cc27b6..13c86ed582 100644 --- a/include/mbedtls/mbedtls_config.h +++ b/include/mbedtls/mbedtls_config.h @@ -3080,7 +3080,7 @@ * If the implementation here is empty, this will effectively disable the * checking of functions' return values. */ -//#define MBEDTLS_CHECK_RETURN +//#define MBEDTLS_CHECK_RETURN __attribute__(__warn_unused_result__) /* PSA options */ /**