diff options
author | Emilia Kasper <emilia@openssl.org> | 2014-08-28 19:45:55 +0200 |
---|---|---|
committer | Emilia Kasper <emilia@openssl.org> | 2014-09-02 15:24:54 +0200 |
commit | 27739e92659d38cdefa21e51b7f52b81a7ac3388 (patch) | |
tree | cfc8ae542fbb9bd2f093f8256eab63849bb51193 /crypto/constant_time_locl.h | |
parent | 157c345175bd4b579918fb2942857ff033b08b41 (diff) |
Make the inline const-time functions static.
"inline" without static is not correct as the compiler may choose to ignore it
and will then either emit an external definition, or expect one.
Reviewed-by: Geoff Thorpe <geoff@openssl.org>
(cherry picked from commit 86f50b36e63275a916b147f9d8764e3c0c060fdb)
Diffstat (limited to 'crypto/constant_time_locl.h')
-rw-r--r-- | crypto/constant_time_locl.h | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/crypto/constant_time_locl.h b/crypto/constant_time_locl.h index 782da6c8b2..0d3acd58b3 100644 --- a/crypto/constant_time_locl.h +++ b/crypto/constant_time_locl.h @@ -81,38 +81,38 @@ static inline unsigned int constant_time_msb(unsigned int a); /* * Returns 0xff..f if a < b and 0 otherwise. */ -inline unsigned int constant_time_lt(unsigned int a, unsigned int b); +static inline unsigned int constant_time_lt(unsigned int a, unsigned int b); /* Convenience method for getting an 8-bit mask. */ -inline unsigned char constant_time_lt_8(unsigned int a, unsigned int b); +static inline unsigned char constant_time_lt_8(unsigned int a, unsigned int b); /* * Returns 0xff..f if a >= b and 0 otherwise. */ -inline unsigned int constant_time_ge(unsigned int a, unsigned int b); +static inline unsigned int constant_time_ge(unsigned int a, unsigned int b); /* Convenience method for getting an 8-bit mask. */ -inline unsigned char constant_time_ge_8(unsigned int a, unsigned int b); +static inline unsigned char constant_time_ge_8(unsigned int a, unsigned int b); /* * Returns 0xff..f if a == 0 and 0 otherwise. */ -inline unsigned int constant_time_is_zero(unsigned int a); +static inline unsigned int constant_time_is_zero(unsigned int a); /* Convenience method for getting an 8-bit mask. */ -inline unsigned char constant_time_is_zero_8(unsigned int a); +static inline unsigned char constant_time_is_zero_8(unsigned int a); /* * Returns 0xff..f if a == b and 0 otherwise. */ -inline unsigned int constant_time_eq(unsigned int a, unsigned int b); +static inline unsigned int constant_time_eq(unsigned int a, unsigned int b); /* Convenience method for getting an 8-bit mask. */ -inline unsigned char constant_time_eq_8(unsigned int a, unsigned int b); +static inline unsigned char constant_time_eq_8(unsigned int a, unsigned int b); static inline unsigned int constant_time_msb(unsigned int a) { return (unsigned int)((int)(a) >> (sizeof(int) * 8 - 1)); } -inline unsigned int constant_time_lt(unsigned int a, unsigned int b) +static inline unsigned int constant_time_lt(unsigned int a, unsigned int b) { unsigned int lt; /* Case 1: msb(a) == msb(b). a < b iff the MSB of a - b is set.*/ @@ -122,12 +122,12 @@ inline unsigned int constant_time_lt(unsigned int a, unsigned int b) return constant_time_msb(lt); } -inline unsigned char constant_time_lt_8(unsigned int a, unsigned int b) +static inline unsigned char constant_time_lt_8(unsigned int a, unsigned int b) { return (unsigned char)(constant_time_lt(a, b)); } -inline unsigned int constant_time_ge(unsigned int a, unsigned int b) +static inline unsigned int constant_time_ge(unsigned int a, unsigned int b) { unsigned int ge; /* Case 1: msb(a) == msb(b). a >= b iff the MSB of a - b is not set.*/ @@ -137,27 +137,27 @@ inline unsigned int constant_time_ge(unsigned int a, unsigned int b) return constant_time_msb(ge); } -inline unsigned char constant_time_ge_8(unsigned int a, unsigned int b) +static inline unsigned char constant_time_ge_8(unsigned int a, unsigned int b) { return (unsigned char)(constant_time_ge(a, b)); } -inline unsigned int constant_time_is_zero(unsigned int a) +static inline unsigned int constant_time_is_zero(unsigned int a) { return constant_time_msb(~a & (a - 1)); } -inline unsigned char constant_time_is_zero_8(unsigned int a) +static inline unsigned char constant_time_is_zero_8(unsigned int a) { return (unsigned char)(constant_time_is_zero(a)); } -inline unsigned int constant_time_eq(unsigned int a, unsigned int b) +static inline unsigned int constant_time_eq(unsigned int a, unsigned int b) { return constant_time_is_zero(a ^ b); } -inline unsigned char constant_time_eq_8(unsigned int a, unsigned int b) +static inline unsigned char constant_time_eq_8(unsigned int a, unsigned int b) { return (unsigned char)(constant_time_eq(a, b)); } |