From d0b66d08bb4beb721b2e6ccff4f4aaf172006126 Mon Sep 17 00:00:00 2001 From: Hanno Becker Date: Fri, 26 Jul 2019 08:07:03 +0100 Subject: [PATCH] Don't use memcpy() for 2-byte copy operation Manual copying is slightly shorter here. --- library/ssl_tls.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/library/ssl_tls.c b/library/ssl_tls.c index 102a13f7d2..3e0552c4ec 100644 --- a/library/ssl_tls.c +++ b/library/ssl_tls.c @@ -5056,13 +5056,11 @@ static int ssl_parse_record_header( mbedtls_ssl_context const *ssl, * Parse and validate record version */ - memcpy( &rec->ver[0], - buf + rec_hdr_version_offset, - rec_hdr_version_len ); - + rec->ver[0] = buf[ rec_hdr_version_offset + 0 ]; + rec->ver[1] = buf[ rec_hdr_version_offset + 1 ]; mbedtls_ssl_read_version( &major_ver, &minor_ver, ssl->conf->transport, - buf + rec_hdr_version_offset ); + &rec->ver[0] ); if( major_ver != ssl->major_ver ) {