diff options
| -rw-r--r-- | crypto/http/http_client.c | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/crypto/http/http_client.c b/crypto/http/http_client.c index 744346adbf..4aba5e7761 100644 --- a/crypto/http/http_client.c +++ b/crypto/http/http_client.c @@ -747,7 +747,7 @@ static ASN1_VALUE *BIO_mem_d2i(BIO *mem, const ASN1_ITEM *it) return resp; } -static BIO *OSSL_HTTP_REQ_CTX_transfer(OSSL_HTTP_REQ_CTX *rctx) +static BIO *ossl_http_req_ctx_transfer(OSSL_HTTP_REQ_CTX *rctx) { int sending = 1; int rv; @@ -777,8 +777,6 @@ static BIO *OSSL_HTTP_REQ_CTX_transfer(OSSL_HTTP_REQ_CTX *rctx) } return NULL; } - if (!BIO_up_ref(rctx->mem)) - return NULL; return rctx->mem; } @@ -790,7 +788,7 @@ ASN1_VALUE *OSSL_HTTP_REQ_CTX_sendreq_d2i(OSSL_HTTP_REQ_CTX *rctx, ERR_raise(ERR_LIB_HTTP, ERR_R_PASSED_NULL_PARAMETER); return NULL; } - return BIO_mem_d2i(OSSL_HTTP_REQ_CTX_transfer(rctx), it); + return BIO_mem_d2i(ossl_http_req_ctx_transfer(rctx), it); } static int update_timeout(int timeout, time_t start_time) @@ -902,7 +900,7 @@ BIO *OSSL_HTTP_transfer(const char *server, const char *port, const char *path, if (rctx == NULL) goto end; - resp = OSSL_HTTP_REQ_CTX_transfer(rctx); + resp = ossl_http_req_ctx_transfer(rctx); if (resp == NULL) { if (rctx->redirection_url != NULL) { if (redirection_url == NULL) @@ -937,14 +935,14 @@ BIO *OSSL_HTTP_transfer(const char *server, const char *port, const char *path, } } } - OSSL_HTTP_REQ_CTX_free(rctx); - /* callback can be used to clean up TLS session */ if (bio_update_fn != NULL - && (*bio_update_fn)(cbio, arg, 0, resp != NULL) == NULL) { - BIO_free(resp); + && (*bio_update_fn)(cbio, arg, 0, resp != NULL) == NULL) resp = NULL; - } + + if (resp != NULL && !BIO_up_ref(resp)) + resp = NULL; + OSSL_HTTP_REQ_CTX_free(rctx); end: /* |