From: Matt Caswell Date: Wed, 31 Jan 2018 13:43:15 +0000 (+0000) Subject: Rename a function to avoid a clash X-Git-Tag: OpenSSL_1_1_1-pre2~78 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=bd74eae1be6c11bbfa95542daa7d59d461464853;p=thirdparty%2Fopenssl.git Rename a function to avoid a clash We already have a constant_time_select() function so, to avoid confusion/clashing we shouldn't have a second one. Reviewed-by: Bernd Edlinger (Merged from https://github.com/openssl/openssl/pull/5105) --- diff --git a/crypto/ec/curve448/constant_time.h b/crypto/ec/curve448/constant_time.h index 84ee02eb681..61389a2b218 100644 --- a/crypto/ec/curve448/constant_time.h +++ b/crypto/ec/curve448/constant_time.h @@ -258,12 +258,12 @@ static ossl_inline void constant_time_lookup(void *RESTRICT out_, * Note that the output is not __restrict__, but if it overlaps either * input, it must be equal and not partially overlap. */ -static ossl_inline void constant_time_select(void *a_, - const void *bFalse_, - const void *bTrue_, - word_t elem_bytes, - mask_t mask, - size_t alignment_bytes) +static ossl_inline void constant_time_select_c448(void *a_, + const void *bFalse_, + const void *bTrue_, + word_t elem_bytes, + mask_t mask, + size_t alignment_bytes) { unsigned char *a = (unsigned char *)a_; const unsigned char *bTrue = (const unsigned char *)bTrue_; diff --git a/crypto/ec/curve448/field.h b/crypto/ec/curve448/field.h index b985d453e78..2b7fc2aa951 100644 --- a/crypto/ec/curve448/field.h +++ b/crypto/ec/curve448/field.h @@ -128,7 +128,7 @@ static ossl_inline void gf_mulw(gf c, const gf a, int32_t w) /* Constant time, x = is_z ? z : y */ static ossl_inline void gf_cond_sel(gf x, const gf y, const gf z, mask_t is_z) { - constant_time_select(x, y, z, sizeof(gf), is_z, 0); + constant_time_select_c448(x, y, z, sizeof(gf), is_z, 0); } /* Constant time, if (neg) x=-x; */