[Skiboot] [PATCH] secvar/test: use mbedtls cflags when building the test binaries

Eric Richter erichte at linux.ibm.com
Fri Oct 2 11:54:19 AEST 2020


The edk2 test file includes some mbedtls files directly, make sure that
those also include the correct mbedtls config file.

Without this, the default config file is used, which conflicts with the
version we build as part of skiboot.

As host libc includes a SIZE_MAX macro, this also changes the SIZE_MAX
macro defined in mbedtls_config.h (needed for some mbedtls functions) to
only be defined if it isn't already.

Signed-off-by: Eric Richter <erichte at linux.ibm.com>
---
 libstb/crypto/mbedtls-config.h    | 2 ++
 libstb/secvar/test/Makefile.check | 4 ++--
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/libstb/crypto/mbedtls-config.h b/libstb/crypto/mbedtls-config.h
index 9560993b..e0c358d2 100644
--- a/libstb/crypto/mbedtls-config.h
+++ b/libstb/crypto/mbedtls-config.h
@@ -86,7 +86,9 @@
 /* Settings to reduce/remove warnings */
 #define MBEDTLS_MPI_WINDOW_SIZE	3	// (max/default is 6) Increase for speed, may introduce warnings
 #define MBEDTLS_MPI_MAX_SIZE	512	// (default is 1024) increase for more bits in user-MPIs
+#ifndef SIZE_MAX
 #define SIZE_MAX		65535	// this might need to be in libc?
+#endif
 
 /* Disableable to mitigate warnings */
 //#define MBEDTLS_ASN1_WRITE_C  // Expects SIZE_MAX
diff --git a/libstb/secvar/test/Makefile.check b/libstb/secvar/test/Makefile.check
index b2cafcf6..94da7dd9 100644
--- a/libstb/secvar/test/Makefile.check
+++ b/libstb/secvar/test/Makefile.check
@@ -42,10 +42,10 @@ HOST_MBEDTLS_CFLAGS+= -Wno-unused-function -Wno-suggest-attribute=const
 $(SECVAR_TEST) : core/test/stubs.o
 
 $(SECVAR_TEST) : % : %.c $(HOST_MBEDTLS_OBJS)
-	$(call Q, HOSTCC ,$(HOSTCC) $(HOSTCFLAGS) -O0 -g -I include -I . -I libfdt -o $@ $< $(HOST_MBEDTLS_OBJS) core/test/stubs.o, $<)
+	$(call Q, HOSTCC ,$(HOSTCC) $(HOSTCFLAGS) $(HOST_MBEDTLS_CFLAGS) -O0 -g -I include -I . -I libfdt -o $@ $< $(HOST_MBEDTLS_OBJS) core/test/stubs.o, $<)
 
 $(SECVAR_TEST:%=%-gcov): %-gcov : %.c % $(HOST_MBEDTLS_OBJS)
-	$(call Q, HOSTCC ,$(HOSTCC) $(HOSTCFLAGS) $(HOSTGCOVCFLAGS) -I include -I . -I libfdt -lgcov -o $@ $< $(HOST_MBEDTLS_OBJS) core/test/stubs.o, $<)
+	$(call Q, HOSTCC ,$(HOSTCC) $(HOSTCFLAGS) $(HOSTGCOVCFLAGS) $(HOST_MBEDTLS_CFLAGS) -I include -I . -I libfdt -lgcov -o $@ $< $(HOST_MBEDTLS_OBJS) core/test/stubs.o, $<)
 
 -include $(wildcard libstb/secvar/test/*.d)
 
-- 
2.21.1



More information about the Skiboot mailing list