[openssl-commits] [openssl] OpenSSL_1_0_2-stable update

Andy Polyakov appro at openssl.org
Sun Mar 26 16:31:08 UTC 2017


The branch OpenSSL_1_0_2-stable has been updated
       via  72c0482d9f29dcd7771a82ddcde15048a4702e8c (commit)
      from  cf5e259d5ba83e62b67ae8bf093f3ef1739812ed (commit)


- Log -----------------------------------------------------------------
commit 72c0482d9f29dcd7771a82ddcde15048a4702e8c
Author: Andy Polyakov <appro at openssl.org>
Date:   Sat Mar 25 10:58:57 2017 +0100

    aes/asm/bsaes-armv7.pl: relax stack alignment requirement.
    
    Even though Apple refers to Procedure Call Standard for ARM Architecture
    (AAPCS), they apparently adhere to custom version that doesn't follow
    stack alignment constraints in the said standard. [Why or why? If it's
    vendor lock-in thing, then it would be like worst spot ever.] And since
    bsaes-armv7 relied on standard alignment, it became problematic to
    execute the code on iOS.
    
    Reviewed-by: Rich Salz <rsalz at openssl.org>
    (cherry picked from commit 0822d41b6d54132df96c02cc6f6fa9b179378351)

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

Summary of changes:
 crypto/aes/asm/bsaes-armv7.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/crypto/aes/asm/bsaes-armv7.pl b/crypto/aes/asm/bsaes-armv7.pl
index 83343e2..70b3f96 100644
--- a/crypto/aes/asm/bsaes-armv7.pl
+++ b/crypto/aes/asm/bsaes-armv7.pl
@@ -1333,7 +1333,7 @@ bsaes_cbc_encrypt:
 	vmov	@XMM[4], at XMM[15]		@ just in case ensure that IV
 	vmov	@XMM[5], at XMM[0]			@ and input are preserved
 	bl	AES_decrypt
-	vld1.8	{@XMM[0]}, [$fp,:64]		@ load result
+	vld1.8	{@XMM[0]}, [$fp]		@ load result
 	veor	@XMM[0], @XMM[0], @XMM[4]	@ ^= IV
 	vmov	@XMM[15], @XMM[5]		@ @XMM[5] holds input
 	vst1.8	{@XMM[0]}, [$rounds]		@ write output


More information about the openssl-commits mailing list