The branch master has been updated
via 70a579ae2f37437a1e02331eeaa84e1b68ba021e (commit)
from 06deb93286ac5f125fc81ddc9260b9de2311c7f3 (commit)
- Log -----------------------------------------------------------------
commit 70a579ae2f37437a1e02331eeaa84e1b68ba021e
Author: Andy Polyakov <[email protected]>
Date: Wed Jul 25 10:29:51 2018 +0200
bn/bn_mod.c: harmonize BN_mod_add_quick with original implementation.
New implementation failed to correctly reset r->neg flag. Spotted by
OSSFuzz.
Reviewed-by: Rich Salz <[email protected]>
(Merged from https://github.com/openssl/openssl/pull/6783)
-----------------------------------------------------------------------
Summary of changes:
crypto/bn/bn_mod.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/crypto/bn/bn_mod.c b/crypto/bn/bn_mod.c
index 463d2d6..d8e2e12 100644
--- a/crypto/bn/bn_mod.c
+++ b/crypto/bn/bn_mod.c
@@ -83,6 +83,7 @@ int bn_mod_add_fixed_top(BIGNUM *r, const BIGNUM *a, const
BIGNUM *b,
((volatile BN_ULONG *)tp)[i] = 0;
}
r->top = mtop;
+ r->neg = 0;
if (tp != storage)
OPENSSL_free(tp);
_____
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits
