commit 5969a4b5e0fb2ab017b05b43d5341a117ce46825
author Gilles Peskine <Gilles.Peskine@arm.com> Mon Jun 27 23:59:20 2022 +0200
committer Gilles Peskine <Gilles.Peskine@arm.com> Mon Jun 27 23:59:53 2022 +0200
tree bb1cd7d73f6c316d3d8aa2e604d9720421be64e4
parent 6194053feb2a14a42ffaaa8042f39ab025b754f0

Don't call memcpy(NULL, 0) which has undefined behavior

Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>

tests/suites/test_suite_asn1write.function

diff --git a/tests/suites/test_suite_asn1write.function b/tests/suites/test_suite_asn1write.function
index 8c260f7..4ed8644 100644
--- a/tests/suites/test_suite_asn1write.function
+++ b/tests/suites/test_suite_asn1write.function
@@ -417,9 +417,12 @@
 
 #if defined(MBEDTLS_ASN1_PARSE_C)
     ASSERT_ALLOC( masked_bitstring, byte_length );
-    memcpy( masked_bitstring, bitstring->x, byte_length );
-    if( bits % 8 != 0 )
-        masked_bitstring[byte_length - 1] &= ~( 0xff >> ( bits % 8 ) );
+    if( byte_length != 0 )
+    {
+        memcpy( masked_bitstring, bitstring->x, byte_length );
+        if( bits % 8 != 0 )
+            masked_bitstring[byte_length - 1] &= ~( 0xff >> ( bits % 8 ) );
+    }
     size_t value_bits = bits;
     if( is_named )
     {