commit 57b684f9d1032ff437cca48c73f14d7bea7e244d
author Hanno Becker <hanno.becker@arm.com> Tue Dec 18 12:50:02 2018 +0000
committer Hanno Becker <hanno.becker@arm.com> Tue Dec 18 13:00:48 2018 +0000
tree 096ab134a9e781e0f647322496b699030d9abb4a
parent 5edcfa529fed02a77b529e533e218520d4f97799

Add separate test for xxx_free() functions in ECP module

tests/suites/test_suite_ecp.data

diff --git a/tests/suites/test_suite_ecp.data b/tests/suites/test_suite_ecp.data
index 1b46306..30d5ec6 100644
--- a/tests/suites/test_suite_ecp.data
+++ b/tests/suites/test_suite_ecp.data
@@ -1,3 +1,6 @@
+ECP valid params
+ecp_valid_param:
+
 ECP invalid params
 ecp_invalid_param:
 

tests/suites/test_suite_ecp.function

diff --git a/tests/suites/test_suite_ecp.function b/tests/suites/test_suite_ecp.function
index 94ba8cb..49ab870 100644
--- a/tests/suites/test_suite_ecp.function
+++ b/tests/suites/test_suite_ecp.function
@@ -13,6 +13,22 @@
  * END_DEPENDENCIES
  */
 
+/* BEGIN_CASE */
+void ecp_valid_param( )
+{
+    TEST_VALID_PARAM( mbedtls_ecp_point_free( NULL ) );
+    TEST_VALID_PARAM( mbedtls_ecp_keypair_free( NULL ) );
+    TEST_VALID_PARAM( mbedtls_ecp_point_free( NULL ) );
+
+#if defined(MBEDTLS_ECP_RESTARTABLE)
+    TEST_VALID_PARAM( mbedtls_ecp_restart_free( NULL ) );
+#endif /* MBEDTLS_ECP_RESTARTABLE */
+
+exit:
+    return;
+}
+/* END_CASE */
+
 /* BEGIN_CASE depends_on:MBEDTLS_CHECK_PARAMS:!MBEDTLS_PARAM_FAILED_ALT */
 void ecp_invalid_param( )
 {
@@ -32,13 +48,8 @@
     TEST_INVALID_PARAM( mbedtls_ecp_keypair_init( NULL ) );
     TEST_INVALID_PARAM( mbedtls_ecp_point_init( NULL ) );
 
-    TEST_VALID_PARAM( mbedtls_ecp_point_free( NULL ) );
-    TEST_VALID_PARAM( mbedtls_ecp_keypair_free( NULL ) );
-    TEST_VALID_PARAM( mbedtls_ecp_point_free( NULL ) );
-
 #if defined(MBEDTLS_ECP_RESTARTABLE)
     TEST_INVALID_PARAM( mbedtls_ecp_restart_init( NULL ) );
-    TEST_VALID_PARAM( mbedtls_ecp_restart_free( NULL ) );
 #endif /* MBEDTLS_ECP_RESTARTABLE */
 
     TEST_INVALID_PARAM_RET( MBEDTLS_ERR_ECP_BAD_INPUT_DATA,