commit 53585eeb1765ee61ae28fa548f663b19bb40b841
author Manuel Pégourié-Gonnard <mpg@elzevir.fr> Thu Jun 25 08:52:25 2015 +0200
committer Manuel Pégourié-Gonnard <mpg@elzevir.fr> Thu Jun 25 10:59:57 2015 +0200
tree 2872596156d7c913cd15712f57d3e86b5801b92c
parent 21e1ac205ed3b9e222bb319eff5b48931d9c9897

Remove test DHM params from certs.c

certs.c belongs to the X.509 library, while DHM belongs to the crypto lib.

ChangeLog

diff --git a/ChangeLog b/ChangeLog
index 40b2d26..2ca6d63 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -105,6 +105,7 @@
      (use generic functions from md.h)
    * Removed mbedtls_timing_msleep(). Use mbedtls_net_usleep() or a custom
      waiting function.
+   * Removed test DHM parameters from the test certs module.
    * Removed the PBKDF2 module (use PKCS5).
    * Removed POLARSSL_ERROR_STRERROR_BC (use mbedtls_strerror()).
    * Removed compat-1.2.h (helper for migrating from 1.2 to 1.3).

include/mbedtls/certs.h

diff --git a/include/mbedtls/certs.h b/include/mbedtls/certs.h
index ee4d3a6..d0aa555 100644
--- a/include/mbedtls/certs.h
+++ b/include/mbedtls/certs.h
@@ -93,11 +93,6 @@
 extern const size_t mbedtls_test_cli_key_rsa_len;
 #endif
 
-#if defined(MBEDTLS_DHM_C)
-extern const char   mbedtls_test_dhm_params[];
-extern const size_t mbedtls_test_dhm_params_len;
-#endif
-
 #ifdef __cplusplus
 }
 #endif

library/certs.c

diff --git a/library/certs.c b/library/certs.c
index 012e472..2dd7eba 100644
--- a/library/certs.c
+++ b/library/certs.c
@@ -291,16 +291,6 @@
 #define TEST_CA_CRT_RSA
 #endif /* MBEDTLS_RSA_C */
 
-#if defined(MBEDTLS_DHM_C)
-const char mbedtls_test_dhm_params[] =
-"-----BEGIN DH PARAMETERS-----\r\n"
-"MIGHAoGBAJ419DBEOgmQTzo5qXl5fQcN9TN455wkOL7052HzxxRVMyhYmwQcgJvh\r\n"
-"1sa18fyfR9OiVEMYglOpkqVoGLN7qd5aQNNi5W7/C+VBdHTBJcGZJyyP5B3qcz32\r\n"
-"9mLJKudlVudV0Qxk5qUJaPZ/xupz0NyoVpviuiBOI1gNi8ovSXWzAgEC\r\n"
-"-----END DH PARAMETERS-----\r\n";
-const size_t mbedtls_test_dhm_params_len = sizeof( mbedtls_test_dhm_params );
-#endif
-
 #if defined(MBEDTLS_PEM_PARSE_C)
 /* Concatenation of all available CA certificates */
 const char mbedtls_test_cas_pem[] = TEST_CA_CRT_RSA TEST_CA_CRT_EC;

library/dhm.c

diff --git a/library/dhm.c b/library/dhm.c
index 979fd07..ac6261f 100644
--- a/library/dhm.c
+++ b/library/dhm.c
@@ -578,14 +578,20 @@
 
 #if defined(MBEDTLS_SELF_TEST)
 
-#include "mbedtls/certs.h"
+static const char mbedtls_test_dhm_params[] =
+"-----BEGIN DH PARAMETERS-----\r\n"
+"MIGHAoGBAJ419DBEOgmQTzo5qXl5fQcN9TN455wkOL7052HzxxRVMyhYmwQcgJvh\r\n"
+"1sa18fyfR9OiVEMYglOpkqVoGLN7qd5aQNNi5W7/C+VBdHTBJcGZJyyP5B3qcz32\r\n"
+"9mLJKudlVudV0Qxk5qUJaPZ/xupz0NyoVpviuiBOI1gNi8ovSXWzAgEC\r\n"
+"-----END DH PARAMETERS-----\r\n";
+
+static const size_t mbedtls_test_dhm_params_len = sizeof( mbedtls_test_dhm_params );
 
 /*
  * Checkup routine
  */
 int mbedtls_dhm_self_test( int verbose )
 {
-#if defined(MBEDTLS_CERTS_C)
     int ret;
     mbedtls_dhm_context dhm;
 
@@ -594,8 +600,9 @@
     if( verbose != 0 )
         mbedtls_printf( "  DHM parameter load: " );
 
-    if( ( ret = mbedtls_dhm_parse_dhm( &dhm, (const unsigned char *) mbedtls_test_dhm_params,
-                               mbedtls_test_dhm_params_len ) ) != 0 )
+    if( ( ret = mbedtls_dhm_parse_dhm( &dhm,
+                    (const unsigned char *) mbedtls_test_dhm_params,
+                    mbedtls_test_dhm_params_len ) ) != 0 )
     {
         if( verbose != 0 )
             mbedtls_printf( "failed\n" );
@@ -611,12 +618,6 @@
     mbedtls_dhm_free( &dhm );
 
     return( ret );
-#else
-    if( verbose != 0 )
-        mbedtls_printf( "  DHM parameter load: skipped\n" );
-
-    return( 0 );
-#endif /* MBEDTLS_CERTS_C */
 }
 
 #endif /* MBEDTLS_SELF_TEST */