commit 3652e99100045727731910834d04a9e4e9f85a23
author Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com> Mon Jul 01 12:09:22 2019 +0200
committer Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com> Tue Jul 02 15:13:18 2019 +0200
tree 143d5370ea4b7df3f7d0805bac15a4c4cd65fcce
parent 44b10761cc9825b1847eaf081b460f75ab20b2e3

Add getter function for handshake->resume

This makes the code more readable by having fewer #ifdefs all over the place.

library/ssl_srv.c

diff --git a/library/ssl_srv.c b/library/ssl_srv.c
index 7786186..07bbe2d 100644
--- a/library/ssl_srv.c
+++ b/library/ssl_srv.c
@@ -2643,7 +2643,7 @@
      * It may be already set to 1 by ssl_parse_session_ticket_ext().
      * If not, try looking up session ID in our cache.
      */
-    if( ssl->handshake->resume == 0 &&
+    if( mbedtls_ssl_handshake_get_resume( ssl->handshake ) == 0 &&
 #if defined(MBEDTLS_SSL_RENEGOTIATION)
         ssl->renego_status == MBEDTLS_SSL_INITIAL_HANDSHAKE &&
 #endif
@@ -2657,7 +2657,7 @@
 #endif /* !MBEDTLS_SSL_NO_SESSION_CACHE */
 
 #if !defined(MBEDTLS_SSL_NO_SESSION_RESUMPTION)
-    if( ssl->handshake->resume == 1 )
+    if( mbedtls_ssl_handshake_get_resume( ssl->handshake ) == 1 )
     {
         /*
          * Resuming a session
@@ -2714,10 +2714,8 @@
 
     MBEDTLS_SSL_DEBUG_MSG( 3, ( "server hello, session id len.: %d", n ) );
     MBEDTLS_SSL_DEBUG_BUF( 3,   "server hello, session id", buf + 39, n );
-#if !defined(MBEDTLS_SSL_NO_SESSION_RESUMPTION)
     MBEDTLS_SSL_DEBUG_MSG( 3, ( "%s session has been resumed",
-                   ssl->handshake->resume ? "a" : "no" ) );
-#endif /* !MBEDTLS_SSL_NO_SESSION_RESUMPTION */
+            mbedtls_ssl_handshake_get_resume( ssl->handshake ) ? "a" : "no" ) );
 
     *p++ = (unsigned char)( ssl->session_negotiate->ciphersuite >> 8 );
     *p++ = (unsigned char)( ssl->session_negotiate->ciphersuite      );