KEYGEN: improve error handling
This commit is contained in:
parent
67ec5615c7
commit
bb0207ba03
@ -45,25 +45,23 @@ ntru_create_keypair(
|
|||||||
keypair *pair,
|
keypair *pair,
|
||||||
ntru_context *ctx)
|
ntru_context *ctx)
|
||||||
{
|
{
|
||||||
bool retval = true;
|
bool retval = false;
|
||||||
fmpz_poly_t Fq,
|
fmpz_poly_t Fq,
|
||||||
Fp,
|
Fp,
|
||||||
pub;
|
pub;
|
||||||
|
|
||||||
|
if (!f || !g || !ctx)
|
||||||
|
goto _return;
|
||||||
|
|
||||||
fmpz_poly_init(Fq);
|
fmpz_poly_init(Fq);
|
||||||
fmpz_poly_init(Fp);
|
fmpz_poly_init(Fp);
|
||||||
fmpz_poly_init(pub);
|
fmpz_poly_init(pub);
|
||||||
|
|
||||||
|
if (!poly_inverse_poly_q(f, Fq, ctx))
|
||||||
|
goto _cleanup;
|
||||||
|
|
||||||
if (!poly_inverse_poly_q(f, Fq, ctx)) {
|
if (!poly_inverse_poly_p(f, Fp, ctx))
|
||||||
retval = false;
|
goto _cleanup;
|
||||||
goto cleanup;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!poly_inverse_poly_p(f, Fp, ctx)) {
|
|
||||||
retval = false;
|
|
||||||
goto cleanup;
|
|
||||||
}
|
|
||||||
|
|
||||||
poly_starmultiply(Fq, g, pub, ctx, ctx->q);
|
poly_starmultiply(Fq, g, pub, ctx, ctx->q);
|
||||||
fmpz_poly_scalar_mul_ui(pub, pub, ctx->p);
|
fmpz_poly_scalar_mul_ui(pub, pub, ctx->p);
|
||||||
@ -77,10 +75,13 @@ ntru_create_keypair(
|
|||||||
fmpz_poly_set(pair->priv_inv, Fp);
|
fmpz_poly_set(pair->priv_inv, Fp);
|
||||||
fmpz_poly_set(pair->pub, pub);
|
fmpz_poly_set(pair->pub, pub);
|
||||||
|
|
||||||
cleanup:
|
retval = true;
|
||||||
|
|
||||||
|
_cleanup:
|
||||||
fmpz_poly_clear(Fq);
|
fmpz_poly_clear(Fq);
|
||||||
fmpz_poly_clear(Fp);
|
fmpz_poly_clear(Fp);
|
||||||
fmpz_poly_clear(pub);
|
fmpz_poly_clear(pub);
|
||||||
|
_return:
|
||||||
return retval;
|
return retval;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user