commit 293c3dae6dce53b03b5149f097009c147644e0e3
author Janos Follath <janos.follath@arm.com> Fri May 10 15:53:03 2019 +0100
committer Janos Follath <janos.follath@arm.com> Fri May 10 15:53:03 2019 +0100
tree 638f07743de2a2f0e563e6dea05a8ec41bc4cdbc
parent 6c379b4b8091ede8d28becc6bf36c7a7c59c3b37

Remove redundant memset()

The preceding calloc() already zeroizes that memory area, therfore the
memset() is not necessary. Compilers are likely to optimize this out,
but it still can be confusing to readers.

library/x509_crt.c

diff --git a/library/x509_crt.c b/library/x509_crt.c
index 047e4e6..f58b156 100644
--- a/library/x509_crt.c
+++ b/library/x509_crt.c
@@ -1858,7 +1858,6 @@
                     if( prev_san != NULL )
                         prev_san->next = cur_san;
 
-                    memset( cur_san, 0, sizeof( mbedtls_x509_subject_alternative_name ) );
                     cur_san->type = MBEDTLS_X509_SAN_DNS_NAME;
 
                     memcpy( &cur_san->san.unstructured_name,