commit 069eb7904327a38feae8a0752071dc408a8af55d
author Manuel Pégourié-Gonnard <mpg@elzevir.fr> Wed Sep 10 20:08:29 2014 +0200
committer Paul Bakker <p.j.bakker@polarssl.org> Tue Oct 21 16:30:30 2014 +0200
tree b5f481408e2a28ccb659d97b2e713a044ed73bf9
parent 04c1b4ece1e61e345e41ba40504f4640bd3aa0f1

No memmove: ssl_parse_hello_verify_request()

library/ssl_cli.c

diff --git a/library/ssl_cli.c b/library/ssl_cli.c
index 5742616..cab6741 100644
--- a/library/ssl_cli.c
+++ b/library/ssl_cli.c
@@ -959,7 +959,7 @@
 #if defined(POLARSSL_SSL_PROTO_DTLS)
 static int ssl_parse_hello_verify_request( ssl_context *ssl )
 {
-    const unsigned char *p = ssl->in_msg + 4;
+    const unsigned char *p = ssl->in_msg + ssl_hs_hdr_len( ssl );
     int major_ver, minor_ver;
     unsigned char cookie_len;
 
@@ -1069,8 +1069,6 @@
         return( POLARSSL_ERR_SSL_UNEXPECTED_MESSAGE );
     }
 
-    ssl_hs_rm_dtls_hdr( ssl );
-
 #if defined(POLARSSL_SSL_PROTO_DTLS)
     if( ssl->transport == SSL_TRANSPORT_DATAGRAM )
     {
@@ -1090,6 +1088,8 @@
     }
 #endif /* POLARSSL_SSL_PROTO_DTLS */
 
+    ssl_hs_rm_dtls_hdr( ssl );
+
     if( ssl->in_hslen < 42 ||
         buf[0] != SSL_HS_SERVER_HELLO )
     {