d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

factor(6): Check return values of BN_* functions.

Browse Source 
Coverity Prevent:	CID 4862, 8771, 8772, 8773
This commit is contained in:
uqs 2010-12-20 19:07:56 +00:00
parent 30fb15bb2d
commit 755da7b81e
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
games/factor/factor.c
View File

@ -211,7 +211,8 @@ pr_fact(BIGNUM *val)
bnfact = BN_new(); bnfact = BN_new();
BN_set_word(bnfact, *(fact - 1)); BN_set_word(bnfact, *(fact - 1));
BN_sqr(bnfact, bnfact, ctx); if (!BN_sqr(bnfact, bnfact, ctx))
errx(1, "error in BN_sqr()");
if (BN_cmp(bnfact, val) > 0 || if (BN_cmp(bnfact, val) > 0 ||
BN_is_prime(val, PRIME_CHECKS, BN_is_prime(val, PRIME_CHECKS,
NULL, NULL, NULL) == 1) NULL, NULL, NULL) == 1)
@ -271,7 +272,8 @@ pollard_pminus1(BIGNUM *val)
BN_set_word(rbase, 1); BN_set_word(rbase, 1);
newbase: newbase:
BN_add_word(rbase, 1); if (!BN_add_word(rbase, 1))
errx(1, "error in BN_add_word()");
BN_set_word(i, 2); BN_set_word(i, 2);
BN_copy(base, rbase); BN_copy(base, rbase);
@ -282,7 +284,8 @@ newbase:
BN_copy(x, base); BN_copy(x, base);
BN_sub_word(x, 1); BN_sub_word(x, 1);
BN_gcd(x, x, val, ctx); if (!BN_gcd(x, x, val, ctx))
errx(1, "error in BN_gcd()");
if (!BN_is_one(x)) { if (!BN_is_one(x)) {
if (BN_is_prime(x, PRIME_CHECKS, NULL, NULL, if (BN_is_prime(x, PRIME_CHECKS, NULL, NULL,
@ -303,7 +306,8 @@ newbase:
} }
BN_copy(val, num); BN_copy(val, num);
} }
BN_add_word(i, 1); if (!BN_add_word(i, 1))
errx(1, "error in BN_add_word()");
} }
} }