diff options
| author | Andy Polyakov <appro@openssl.org> | 2018-07-18 15:13:27 +0200 |
|---|---|---|
| committer | Andy Polyakov <appro@openssl.org> | 2018-07-22 15:21:18 +0200 |
| commit | f40e0a342cbca8bb71d0fe3f19e1b4bfd853aff1 (patch) | |
| tree | 1d87f395400349766d8acb9ce6dd49abcd746986 /crypto | |
| parent | 708c28f2f0598af6bccbeb60fb46086784aed7da (diff) | |
bn/bn_intern.c: const-ify bn_set_{static}_words.
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/6738)
Diffstat (limited to 'crypto')
| -rw-r--r-- | crypto/bn/bn_intern.c | 10 | ||||
| -rw-r--r-- | crypto/include/internal/bn_int.h | 4 |
2 files changed, 9 insertions, 5 deletions
diff --git a/crypto/bn/bn_intern.c b/crypto/bn/bn_intern.c index 2524eb2acf..faebdc8e11 100644 --- a/crypto/bn/bn_intern.c +++ b/crypto/bn/bn_intern.c @@ -172,16 +172,20 @@ BN_ULONG *bn_get_words(const BIGNUM *a) return a->d; } -void bn_set_static_words(BIGNUM *a, BN_ULONG *words, int size) +void bn_set_static_words(BIGNUM *a, const BN_ULONG *words, int size) { - a->d = words; + /* + * |const| qualifier omission is compensated by BN_FLG_STATIC_DATA + * flag, which effectively means "read-only data". + */ + a->d = (BN_ULONG *)words; a->dmax = a->top = size; a->neg = 0; a->flags |= BN_FLG_STATIC_DATA; bn_correct_top(a); } -int bn_set_words(BIGNUM *a, BN_ULONG *words, int num_words) +int bn_set_words(BIGNUM *a, const BN_ULONG *words, int num_words) { if (bn_wexpand(a, num_words) == NULL) { BNerr(BN_F_BN_SET_WORDS, ERR_R_MALLOC_FAILURE); diff --git a/crypto/include/internal/bn_int.h b/crypto/include/internal/bn_int.h index f7d37d5827..f592912ca8 100644 --- a/crypto/include/internal/bn_int.h +++ b/crypto/include/internal/bn_int.h @@ -47,7 +47,7 @@ BN_ULONG *bn_get_words(const BIGNUM *a); * Set the internal data words in a to point to words which contains size * elements. The BN_FLG_STATIC_DATA flag is set */ -void bn_set_static_words(BIGNUM *a, BN_ULONG *words, int size); +void bn_set_static_words(BIGNUM *a, const BN_ULONG *words, int size); /* * Copy words into the BIGNUM |a|, reallocating space as necessary. @@ -58,7 +58,7 @@ void bn_set_static_words(BIGNUM *a, BN_ULONG *words, int size); * |num_words| is int because bn_expand2 takes an int. This is an internal * function so we simply trust callers not to pass negative values. */ -int bn_set_words(BIGNUM *a, BN_ULONG *words, int num_words); +int bn_set_words(BIGNUM *a, const BN_ULONG *words, int num_words); /* * Some BIGNUM functions assume most significant limb to be non-zero, which |