From 90fc4832a392f60d2da2f5684d8d20ac8d91cebd Mon Sep 17 00:00:00 2001 From: Andy Polyakov Date: Mon, 21 Nov 2016 15:15:52 +0100 Subject: [PATCH] bn/asm/ppc-mont.pl: signal no-op in 32-bit bit build. The bug was introduced in 80d27cdb84985c697f8fabb7649abf1f54714d13, one too many instructions was removed. It went unnoticed, because new subroutine introduced in previous commit is called in real-life RSA/DSA/DH cases, while original code is called only in rare tests. The bug was caught in test_fuzz. Reviewed-by: Rich Salz --- crypto/bn/asm/ppc-mont.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/crypto/bn/asm/ppc-mont.pl b/crypto/bn/asm/ppc-mont.pl index fdc049ae0e..b747f70132 100644 --- a/crypto/bn/asm/ppc-mont.pl +++ b/crypto/bn/asm/ppc-mont.pl @@ -135,6 +135,7 @@ $code=<<___; .align 5 .bn_mul_mont_int: mr $rp,r3 ; $rp is reassigned + li r3,0 ___ $code.=<<___ if ($BNSZ==4); cmpwi $num,32 ; longer key performance is not better -- 2.25.1