commit 272cc19ab5dcf8a7dc1fc80323e1f4348512c544
author Ashley Duncan <ashes.man@gmail.com> Fri Feb 11 09:57:18 2022 +1300
committer Dave Rodgman <dave.rodgman@arm.com> Fri Feb 24 16:02:25 2023 +0000
tree 60a3e4a0ab674032187f11e56303a55f30078784
parent 4dcc08c244b942ae12683dca64e023b08fb5f44d

Fixed undefined behavior in ssl_read if buf parameter is NULL.

Signed-off-by: Ashley Duncan <ashes.man@gmail.com>

library/ssl_msg.c

diff --git a/library/ssl_msg.c b/library/ssl_msg.c
index a38e764..8d35c9c 100644
--- a/library/ssl_msg.c
+++ b/library/ssl_msg.c
@@ -5429,8 +5429,10 @@
     n = (len < ssl->in_msglen)
         ? len : ssl->in_msglen;
 
-    memcpy(buf, ssl->in_offt, n);
-    ssl->in_msglen -= n;
+    if (buf) {
+        memcpy(buf, ssl->in_offt, n);
+        ssl->in_msglen -= n;
+    }
 
     /* Zeroising the plaintext buffer to erase unused application data
        from the memory. */