On Tue, 18 Jul 2017 20:33:54 +0200
[email protected] (Niels Möller) wrote:

> Sergei Trofimovich <[email protected]> writes:
> 
> > Attaching a proof-of-concept patch nettle-3.3-noclobber.patch
> > which fixes tests on sparc.  
> 
> Can you try the below patch, which reorders the multiplies, with no
> change to allocation?

This patch also works. Thank you!

> Regards,
> /Niels
> 
> diff --git a/ecc-add-eh.c b/ecc-add-eh.c
> index a16be4c..c07ff49 100644
> --- a/ecc-add-eh.c
> +++ b/ecc-add-eh.c
> @@ -98,8 +98,8 @@ ecc_add_eh (const struct ecc_curve *ecc,
>    ecc_modp_mul (ecc, x3, B, z1);
>  
>    /* y3 */
> -  ecc_modp_mul (ecc, B, F, C); /* ! */
> -  ecc_modp_mul (ecc, y3, B, z1);
> +  ecc_modp_mul (ecc, B, F, z1); /* ! */
> +  ecc_modp_mul (ecc, y3, B, C); /* Clobbers z1 in case r == p. */
>  
>    /* z3 */
>    ecc_modp_mul (ecc, B, F, G);
> 
> -- 
> Niels Möller. PGP-encrypted email is preferred. Keyid 368C6677.
> Internet email is subject to wholesale government surveillance.


-- 

  Sergei

Attachment: pgpoq99Qq2enr.pgp
Description: Цифровая подпись OpenPGP

_______________________________________________
nettle-bugs mailing list
[email protected]
http://lists.lysator.liu.se/mailman/listinfo/nettle-bugs

Reply via email to