Skip to content

Commit fc7aa1d

Browse files
committed
Don't ignore the set padding return code
1 parent 0792f93 commit fc7aa1d

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

RSA.xs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -968,7 +968,7 @@ sign(p_rsa, text_SV)
968968
CHECK_OPEN_SSL(ctx);
969969
CHECK_OPEN_SSL(EVP_PKEY_sign_init(ctx));
970970
/* FIXME: Issue setting padding in some cases */
971-
EVP_PKEY_CTX_set_rsa_padding(ctx, p_rsa->padding);
971+
CHECK_OPEN_SSL(EVP_PKEY_CTX_set_rsa_padding(ctx, p_rsa->padding) > 0);
972972

973973
EVP_MD* md = get_md_bynid(p_rsa->hashMode);
974974
CHECK_OPEN_SSL(md != NULL);
@@ -1025,7 +1025,7 @@ PPCODE:
10251025
CHECK_OPEN_SSL(ctx);
10261026
CHECK_OPEN_SSL(EVP_PKEY_verify_init(ctx) == 1);
10271027
/* FIXME: Issue setting padding in some cases */
1028-
EVP_PKEY_CTX_set_rsa_padding(ctx, p_rsa->padding);
1028+
CHECK_OPEN_SSL(EVP_PKEY_CTX_set_rsa_padding(ctx, p_rsa->padding) > 0);
10291029

10301030
EVP_MD* md = get_md_bynid(p_rsa->hashMode);
10311031
CHECK_OPEN_SSL(md != NULL);

0 commit comments

Comments
 (0)