commit 0fa3fc80bcef81e29329a3b59544313c885c6c29
author Ronald Cron <ronald.cron@arm.com> Wed Apr 28 08:33:34 2021 +0200
committer GitHub <noreply@github.com> Wed Apr 28 08:33:34 2021 +0200
tree c2bb3e730499614ca97a8719ffbd62a6d2d05b6f
parent b5939e814e836a4c276ec98feb13c1d7cb0e969d
parent 46266670e9c2eb1fd8072c523dd1dbbe2b1050bb

Merge pull request #4432 from daverodgman/fix_missing_parenthesis_2x

[backport 2.x] Add missing parenthesis when MBEDTLS_ECP_NORMALIZE_MXZ_ALT is declared

ChangeLog.d/add-missing-parenthesis.txt

diff --git a/ChangeLog.d/add-missing-parenthesis.txt b/ChangeLog.d/add-missing-parenthesis.txt
new file mode 100644
index 0000000..9576ff3
--- /dev/null
+++ b/ChangeLog.d/add-missing-parenthesis.txt
@@ -0,0 +1,3 @@
+Bugfix
+   * Fix a compilation error when MBEDTLS_ECP_RANDOMIZE_MXZ_ALT is
+     defined. Fixes #4217.

library/ecp.c

diff --git a/library/ecp.c b/library/ecp.c
index ab956e5..6b29415 100644
--- a/library/ecp.c
+++ b/library/ecp.c
@@ -2477,7 +2477,7 @@
 {
 #if defined(MBEDTLS_ECP_RANDOMIZE_MXZ_ALT)
     if( mbedtls_internal_ecp_grp_capable( grp ) )
-        return( mbedtls_internal_ecp_randomize_mxz( grp, P, f_rng, p_rng );
+        return( mbedtls_internal_ecp_randomize_mxz( grp, P, f_rng, p_rng ) );
 #endif /* MBEDTLS_ECP_RANDOMIZE_MXZ_ALT */
 
 #if defined(MBEDTLS_ECP_NO_FALLBACK) && defined(MBEDTLS_ECP_RANDOMIZE_MXZ_ALT)