diff options
| author | Jiasheng Jiang <jiasheng@iscas.ac.cn> | 2022-07-01 16:23:58 +0800 |
|---|---|---|
| committer | Todd Short <todd.short@me.com> | 2022-07-08 13:05:03 -0400 |
| commit | 12e488367d34657a5c0e1bc322e66c48463d2a0c (patch) | |
| tree | 508182842cfdcc65a3b10758ebc0617d0156a2e4 /ssl/tls_srp.c | |
| parent | 65523758e546fcef0f930e5f8878ef51d174dbc8 (diff) | |
ssl/tls_srp.c: Add check for BN_dup
As the potential failure of the BN_dup,
it should be better to check the return value
in order to guarantee the success.
Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
Reviewed-by: Todd Short <todd.short@me.com>
(Merged from https://github.com/openssl/openssl/pull/18699)
Diffstat (limited to 'ssl/tls_srp.c')
| -rw-r--r-- | ssl/tls_srp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ssl/tls_srp.c b/ssl/tls_srp.c index e69ddfe9d5..a4ec363cfb 100644 --- a/ssl/tls_srp.c +++ b/ssl/tls_srp.c @@ -238,7 +238,7 @@ int SSL_set_srp_server_param_pw(SSL *s, const char *user, const char *pass, BN_clear_free(s->srp_ctx.s); s->srp_ctx.s = NULL; if (!SRP_create_verifier_BN_ex(user, pass, &s->srp_ctx.s, &s->srp_ctx.v, - GN->N, GN->g, s->ctx->libctx, + s->srp_ctx.N, s->srp_ctx.g, s->ctx->libctx, s->ctx->propq)) return -1; |