[openssl-commits] [openssl] OpenSSL_1_0_1-stable update

[Kurt Roeckx]

The branch OpenSSL_1_0_1-stable has been updated
       via  eadc81e7dd3fde473a9e38a57b4c29cf6b699110 (commit)
      from  a2fcab9978a0905c4286051993da63329fda8a19 (commit)


- Log -----------------------------------------------------------------
commit eadc81e7dd3fde473a9e38a57b4c29cf6b699110
Author: Andy Polyakov <appro at openssl.org>
Date:   Sat Feb 21 13:51:56 2015 +0100

    Avoid reading an unused byte after the buffer
    
    Other curves don't have this problem.
    
    Reviewed-by: Kurt Roeckx <kurt at roeckx.be>
    Reviewed-by: Emilia Käsper <emilia at openssl.org>
    (cherry picked from commit 9fbbdd73c58c29dc46cc314f7165e45e6d43fd60)

-----------------------------------------------------------------------

Summary of changes:
 crypto/ec/ecp_nistp224.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/crypto/ec/ecp_nistp224.c b/crypto/ec/ecp_nistp224.c
index 9a59ef0..ed09f97 100644
--- a/crypto/ec/ecp_nistp224.c
+++ b/crypto/ec/ecp_nistp224.c
@@ -321,7 +321,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)