diff options
| author | Kazuki Yamaguchi <k@rhe.jp> | 2022-09-02 18:14:57 +0900 |
|---|---|---|
| committer | Kazuki Yamaguchi <k@rhe.jp> | 2022-09-02 18:14:57 +0900 |
| commit | 3992b6f208987de3ec86aa5745765a992edce236 (patch) | |
| tree | 8c1fcb0dfe1fffb2231a97be75c82cdea24d2bc2 /ext/openssl/ossl_pkey.c | |
| parent | bff4080091f1e468418b0a8625174f234922210c (diff) | |
| download | ruby-openssl-3992b6f208987de3ec86aa5745765a992edce236.tar.gz | |
pkey: clear error queue before each OSSL_DECODER_from_bio() call
Fix potential error queue leak.
Diffstat (limited to 'ext/openssl/ossl_pkey.c')
| -rw-r--r-- | ext/openssl/ossl_pkey.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/ext/openssl/ossl_pkey.c b/ext/openssl/ossl_pkey.c index 2a4835a2..63309669 100644 --- a/ext/openssl/ossl_pkey.c +++ b/ext/openssl/ossl_pkey.c @@ -99,17 +99,20 @@ ossl_pkey_read_generic(BIO *bio, VALUE pass) /* First check DER */ if (OSSL_DECODER_from_bio(dctx, bio) == 1) goto out; + OSSL_BIO_reset(bio); /* Then check PEM; multiple OSSL_DECODER_from_bio() calls may be needed */ - OSSL_BIO_reset(bio); if (OSSL_DECODER_CTX_set_input_type(dctx, "PEM") != 1) goto out; - while (OSSL_DECODER_from_bio(dctx, bio) != 1) { - if (BIO_eof(bio)) + while (1) { + if (OSSL_DECODER_from_bio(dctx, bio) == 1) goto out; + if (BIO_eof(bio)) + break; pos2 = BIO_tell(bio); if (pos2 < 0 || pos2 <= pos) - goto out; + break; + ossl_clear_error(); pos = pos2; } |