commit 3ce9b900d262149518f2d3222f6a11c8542f629a
author Gilles Peskine <Gilles.Peskine@arm.com> Sat Jan 06 01:34:21 2018 +0100
committer Gilles Peskine <Gilles.Peskine@arm.com> Tue Apr 24 09:36:36 2018 +0200
tree 7b55470fbed58e3c566b4142edc87445ae6ecf5d
parent 3665f1db9db40b5ff0ea5a10d91a0e899f432e9d

ssl_write_server_key_exchange refactor: remove dig_signed_len

Simplify the redundant varaible dig_signed_len away.

This is in preparation for splitting ssl_write_server_key_exchange
into several functions.

library/ssl_srv.c

diff --git a/library/ssl_srv.c b/library/ssl_srv.c
index ac9e6a5..daebb9e 100644
--- a/library/ssl_srv.c
+++ b/library/ssl_srv.c
@@ -2836,8 +2836,7 @@
 #if defined(MBEDTLS_KEY_EXCHANGE__SOME_PFS__ENABLED)
     size_t len;
 #if defined(MBEDTLS_KEY_EXCHANGE__WITH_SERVER_SIGNATURE__ENABLED)
-    unsigned char *dig_signed = p;
-    size_t dig_signed_len = 0;
+    unsigned char *dig_signed = NULL;
 #endif /* MBEDTLS_KEY_EXCHANGE__WITH_SERVER_SIGNATURE__ENABLED */
 #endif /* MBEDTLS_KEY_EXCHANGE__SOME_PFS__ENABLED */
 
@@ -2961,7 +2960,6 @@
 
 #if defined(MBEDTLS_KEY_EXCHANGE__WITH_SERVER_SIGNATURE__ENABLED)        
         dig_signed = p;
-        dig_signed_len = len;
 #endif
 
         p += len;
@@ -3022,8 +3020,7 @@
         }
 
 #if defined(MBEDTLS_KEY_EXCHANGE__WITH_SERVER_SIGNATURE__ENABLED)
-        dig_signed     = p;
-        dig_signed_len = len;
+        dig_signed = p;
 #endif
 
         p += len;
@@ -3041,6 +3038,7 @@
 #if defined(MBEDTLS_KEY_EXCHANGE__WITH_SERVER_SIGNATURE__ENABLED)
     if( mbedtls_ssl_ciphersuite_uses_server_signature( ciphersuite_info ) )
     {
+        size_t dig_signed_len = p - dig_signed;
         size_t signature_len = 0;
         unsigned int hashlen = 0;
         unsigned char hash[MBEDTLS_MD_MAX_SIZE];