Commit 6a815969 authored by Andy Polyakov's avatar Andy Polyakov
Browse files

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: default avatarRich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/6810)

(cherry picked from commit 70a579ae)
parent 83325a68

crypto/bn/bn_mod.c

+1 −0
Original line number Diff line number Diff line
@@ -197,6 +197,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);