Don't set the two top bits to one when generating a random number < q.:wq
authorUlf Möller <ulf@openssl.org>
Wed, 13 Sep 2000 01:45:54 +0000 (01:45 +0000)
committerUlf Möller <ulf@openssl.org>
Wed, 13 Sep 2000 01:45:54 +0000 (01:45 +0000)
crypto/dsa/dsa_key.c
crypto/dsa/dsa_ossl.c

index 5aef2d5..af3c56d 100644 (file)
@@ -84,7 +84,7 @@ int DSA_generate_key(DSA *dsa)
        i=BN_num_bits(dsa->q);
        for (;;)
                {
-               if (!BN_rand(priv_key,i,1,0))
+               if (!BN_rand(priv_key,i,0,0))
                        goto err;
                if (BN_cmp(priv_key,dsa->q) >= 0)
                        BN_sub(priv_key,priv_key,dsa->q);
index b51cf6a..0943565 100644 (file)
@@ -181,7 +181,7 @@ static int dsa_sign_setup(DSA *dsa, BN_CTX *ctx_in, BIGNUM **kinvp, BIGNUM **rp)
        /* Get random k */
        for (;;)
                {
-               if (!BN_rand(&k, BN_num_bits(dsa->q), 1, 0)) goto err;
+               if (!BN_rand(&k, BN_num_bits(dsa->q), 0, 0)) goto err;
                if (BN_cmp(&k,dsa->q) >= 0)
                        BN_sub(&k,&k,dsa->q);
                if (!BN_is_zero(&k)) break;