commit cad11ada7f7d0b79ac1a49d2d9e0484ce42613a1
author Deomid rojer Ryabkov <rojer@rojer.me> Sat Jan 18 15:58:57 2025 +0200
committer Deomid rojer Ryabkov <rojer@rojer.me> Sat Jan 18 15:59:29 2025 +0200
tree a02539e705ed4c9f3cba847a03bf99c52916a362
parent 5f7c2c21825518d87912aafc8ad5bda5ad0f320b

Review comments

Signed-off-by: Deomid rojer Ryabkov <rojer@rojer.me>

library/ssl_msg.c

diff --git a/library/ssl_msg.c b/library/ssl_msg.c
index 1c548ec..d0b755d 100644
--- a/library/ssl_msg.c
+++ b/library/ssl_msg.c
@@ -3298,15 +3298,14 @@
     {
         int ret;
         const size_t hs_remain = ssl->in_hslen - ssl->in_hsfraglen;
-        const size_t msg_hslen = (hs_remain <= ssl->in_msglen ? hs_remain : ssl->in_msglen);
-
         MBEDTLS_SSL_DEBUG_MSG(3,
                               ("handshake fragment: %" MBEDTLS_PRINTF_SIZET " .. %"
                                MBEDTLS_PRINTF_SIZET " of %"
                                MBEDTLS_PRINTF_SIZET " msglen %" MBEDTLS_PRINTF_SIZET,
-                               ssl->in_hsfraglen, ssl->in_hsfraglen + msg_hslen,
+                               ssl->in_hsfraglen,
+                               ssl->in_hsfraglen +
+                               (hs_remain <= ssl->in_msglen ? hs_remain : ssl->in_msglen),
                                ssl->in_hslen, ssl->in_msglen));
-        (void) msg_hslen;
         if (ssl->in_msglen < hs_remain) {
             ssl->in_hsfraglen += ssl->in_msglen;
             ssl->in_hdr = ssl->in_msg + ssl->in_msglen;
@@ -5424,7 +5423,7 @@
     if (ssl->conf->transport == MBEDTLS_SSL_TRANSPORT_DATAGRAM) {
         ssl->in_hdr = ssl->in_buf;
     } else
-#endif
+#endif  /* MBEDTLS_SSL_PROTO_DTLS */
     {
         ssl->in_hdr = ssl->in_buf + MBEDTLS_SSL_SEQUENCE_NUMBER_LEN;
     }