commit 4905e6c4e7277dc9da502996aec8124193161688
author Gilles Peskine <Gilles.Peskine@arm.com> Mon Dec 04 17:29:13 2017 +0100
committer Gilles Peskine <Gilles.Peskine@arm.com> Mon Dec 04 17:29:13 2017 +0100
tree 91d676d8154f1d8e1773583a37d2ee94e99e83d8
parent 6e206364d94d4b16e91e3840550767d15097f7dd
parent 046fff12fab56996c406e9f08206fca8f62110e0

Merge branch 'pr_1045' into mbedtls-1.3

ChangeLog

diff --git a/ChangeLog b/ChangeLog
index 0c9358f..70a003d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -37,6 +37,7 @@
      POLARSSL_SSL_DISABLE_RENEGOTIATION is set. Found by erja-gp.
    * Add a check for invalid private parameters in ecdsa_sign.
      Reported by Yolan Romailler.
+   * Fix word size check in in pk.c to not depend on MBEDTLS_HAVE_INT64.
 
 Changes
    * Extend cert_write example program by options to set the CRT version

library/pk.c

diff --git a/library/pk.c b/library/pk.c
index fc036d2..8bcba58 100644
--- a/library/pk.c
+++ b/library/pk.c
@@ -30,8 +30,6 @@
 #include "polarssl/pk.h"
 #include "polarssl/pk_wrap.h"
 
-#include "polarssl/bignum.h"
-
 #if defined(POLARSSL_RSA_C)
 #include "polarssl/rsa.h"
 #endif
@@ -43,6 +41,7 @@
 #endif
 
 #include <limits.h>
+#include <stdint.h>
 
 /* Implementation that should never be optimized out by the compiler */
 static void polarssl_zeroize( void *v, size_t n ) {
@@ -212,10 +211,10 @@
         int ret;
         const pk_rsassa_pss_options *pss_opts;
 
-#if defined(POLARSSL_HAVE_INT64)
+#if SIZE_MAX > UINT_MAX
         if( md_alg == POLARSSL_MD_NONE && UINT_MAX < hash_len )
             return( POLARSSL_ERR_PK_BAD_INPUT_DATA );
-#endif /* POLARSSL_HAVE_INT64 */
+#endif /* SIZE_MAX > UINT_MAX */
 
         if( options == NULL )
             return( POLARSSL_ERR_PK_BAD_INPUT_DATA );

library/pk_wrap.c

diff --git a/library/pk_wrap.c b/library/pk_wrap.c
index ceaaad1..9da59ec 100644
--- a/library/pk_wrap.c
+++ b/library/pk_wrap.c
@@ -31,7 +31,6 @@
 
 /* Even if RSA not activated, for the sake of RSA-alt */
 #include "polarssl/rsa.h"
-#include "polarssl/bignum.h"
 
 #include <string.h>
 
@@ -52,6 +51,7 @@
 #endif
 
 #include <limits.h>
+#include <stdint.h>
 
 /* Implementation that should never be optimized out by the compiler */
 static void polarssl_zeroize( void *v, size_t n ) {
@@ -76,10 +76,10 @@
 {
     int ret;
 
-#if defined(POLARSSL_HAVE_INT64)
+#if SIZE_MAX > UINT_MAX
     if( md_alg == POLARSSL_MD_NONE && UINT_MAX < hash_len )
         return( POLARSSL_ERR_PK_BAD_INPUT_DATA );
-#endif /* POLARSSL_HAVE_INT64 */
+#endif /* SIZE_MAX > UINT_MAX */
 
     if( sig_len < ((rsa_context *) ctx)->len )
         return( POLARSSL_ERR_RSA_VERIFY_FAILED );
@@ -100,10 +100,10 @@
                    unsigned char *sig, size_t *sig_len,
                    int (*f_rng)(void *, unsigned char *, size_t), void *p_rng )
 {
-#if defined(POLARSSL_HAVE_INT64)
+#if SIZE_MAX > UINT_MAX
     if( md_alg == POLARSSL_MD_NONE && UINT_MAX < hash_len )
         return( POLARSSL_ERR_PK_BAD_INPUT_DATA );
-#endif /* POLARSSL_HAVE_INT64 */
+#endif /* SIZE_MAX > UINT_MAX */
 
     *sig_len = ((rsa_context *) ctx)->len;
 
@@ -424,10 +424,10 @@
 {
     rsa_alt_context *rsa_alt = (rsa_alt_context *) ctx;
 
-#if defined(POLARSSL_HAVE_INT64)
+#if SIZE_MAX > UINT_MAX
     if( UINT_MAX < hash_len )
         return( POLARSSL_ERR_PK_BAD_INPUT_DATA );
-#endif /* POLARSSL_HAVE_INT64 */
+#endif /* SIZE_MAX > UINT_MAX */
 
     *sig_len = rsa_alt->key_len_func( rsa_alt->key );