diff options
author | Andy Polyakov <appro@openssl.org> | 2015-02-21 13:51:56 +0100 |
---|---|---|
committer | Kurt Roeckx <kurt@roeckx.be> | 2015-03-14 18:23:41 +0100 |
commit | 9fbbdd73c58c29dc46cc314f7165e45e6d43fd60 (patch) | |
tree | fe4d6b33df808bb60a5c5459ce6d526c9a26b18c /crypto/ec | |
parent | 8b37e5c14f0eddb10c7f91ef91004622d90ef361 (diff) | |
download | openssl-9fbbdd73c58c29dc46cc314f7165e45e6d43fd60.tar.gz |
Avoid reading an unused byte after the buffer
Other curves don't have this problem.
Reviewed-by: Kurt Roeckx <kurt@roeckx.be>
Reviewed-by: Emilia Käsper <emilia@openssl.org>
Diffstat (limited to 'crypto/ec')
-rw-r--r-- | crypto/ec/ecp_nistp224.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/ec/ecp_nistp224.c b/crypto/ec/ecp_nistp224.c index ffb50d848c..a0c7bec5db 100644 --- a/crypto/ec/ecp_nistp224.c +++ b/crypto/ec/ecp_nistp224.c @@ -316,7 +316,7 @@ static void bin28_to_felem(felem out, const u8 in[28]) out[0] = *((const uint64_t *)(in)) & 0x00ffffffffffffff; out[1] = (*((const uint64_t *)(in + 7))) & 0x00ffffffffffffff; out[2] = (*((const uint64_t *)(in + 14))) & 0x00ffffffffffffff; - out[3] = (*((const uint64_t *)(in + 21))) & 0x00ffffffffffffff; + out[3] = (*((const uint64_t *)(in+20))) >> 8; } static void felem_to_bin28(u8 out[28], const felem in) |