From 35007df3988a0c3d3bdccd7755fb3f1846bfa46c Mon Sep 17 00:00:00 2001 From: Simon Butcher Date: Sun, 22 Jul 2018 14:43:39 +0100 Subject: [PATCH 1/2] Increase the memory buffer size for ssl_server2.c Newer features in the library have increased the overall RAM usage of the library, when all features are enabled. ssl_server2.c, with all features enabled was running out of memory for the ssl-opt.sh test 'Authentication: client max_int chain, server required'. This commit increases the memory buffer allocation for ssl_server2.c to allow the test to work with all features enabled. --- programs/ssl/ssl_server2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/programs/ssl/ssl_server2.c b/programs/ssl/ssl_server2.c index a770f1b89..77843ab73 100644 --- a/programs/ssl/ssl_server2.c +++ b/programs/ssl/ssl_server2.c @@ -904,7 +904,7 @@ int main( int argc, char *argv[] ) const char *alpn_list[ALPN_LIST_SIZE]; #endif #if defined(MBEDTLS_MEMORY_BUFFER_ALLOC_C) - unsigned char alloc_buf[100000]; + unsigned char alloc_buf[120000]; #endif int i; From 437f96e3a42d69133e18a2fd6db88634525cc235 Mon Sep 17 00:00:00 2001 From: Simon Butcher Date: Mon, 23 Jul 2018 14:26:09 +0100 Subject: [PATCH 2/2] Add a macro to define the memory size in ssl_server2.c When MBEDTLS_MEMORY_BUFFER_ALLOC_C was defined, the sample ssl_server2.c was using its own memory buffer for memory allocated by the library. The memory used wasn't obvious, so this adds a macro for the memory buffer allocated to make the allocated memory size more obvious and hence easier to configure. --- programs/ssl/ssl_server2.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/programs/ssl/ssl_server2.c b/programs/ssl/ssl_server2.c index 77843ab73..adc2ab1e5 100644 --- a/programs/ssl/ssl_server2.c +++ b/programs/ssl/ssl_server2.c @@ -97,6 +97,10 @@ int main( void ) #include #endif +/* Size of memory to be allocated for the heap, when using the library's memory + * management and MBEDTLS_MEMORY_BUFFER_ALLOC_C is enabled. */ +#define MEMORY_HEAP_SIZE 120000 + #define DFL_SERVER_ADDR NULL #define DFL_SERVER_PORT "4433" #define DFL_RESPONSE_SIZE -1 @@ -904,7 +908,7 @@ int main( int argc, char *argv[] ) const char *alpn_list[ALPN_LIST_SIZE]; #endif #if defined(MBEDTLS_MEMORY_BUFFER_ALLOC_C) - unsigned char alloc_buf[120000]; + unsigned char alloc_buf[MEMORY_HEAP_SIZE]; #endif int i;