commit 3e7481e6a20e1c1ca10c6c81bcfe372311582eb9
author Ronald Cron <ronald.cron@arm.com> Wed Apr 28 08:35:00 2021 +0200
committer GitHub <noreply@github.com> Wed Apr 28 08:35:00 2021 +0200
tree e379edce8a2ed398a0fe347d0a4f3a32e29fd1ab
parent 12f93f4fc25055163b42cd91c215ff2c594f036c
parent 0c37b4f82601e2c93ba75acbd75624bc03e4af66

Merge pull request #4219 from stevew817/fix_missing_parenthesis

Add missing parenthesis when MBEDTLS_ECP_NORMALIZE_MXZ_ALT is declared

@mpg comment has been addressed thus this can be merged.

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 4c0d28e..194e448 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)