From ca99e40290031e6a7c9537fc884e45e62858bd73 Mon Sep 17 00:00:00 2001 From: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com> Date: Wed, 28 Jan 2026 14:49:40 +0000 Subject: [PATCH] fix memory leak. Signed-off-by: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com> --- sanitizers/suppressions/lsan.supp | 2 +- src/tests/libxrpl/net/HTTPClient.cpp | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/sanitizers/suppressions/lsan.supp b/sanitizers/suppressions/lsan.supp index 6d3a040b86..06807e5bbf 100644 --- a/sanitizers/suppressions/lsan.supp +++ b/sanitizers/suppressions/lsan.supp @@ -4,7 +4,7 @@ # These are false positives from Boost.Asio SSL internals that use OpenSSL BIO structures. # The BIO structures are managed by OpenSSL's internal reference counting and freed at process exit. -leak:boost::asio +#leak:boost::asio #leak:boost/asio # OpenSSL BIO memory is managed internally and freed at process exit diff --git a/src/tests/libxrpl/net/HTTPClient.cpp b/src/tests/libxrpl/net/HTTPClient.cpp index c1906d8c78..2f063102db 100644 --- a/src/tests/libxrpl/net/HTTPClient.cpp +++ b/src/tests/libxrpl/net/HTTPClient.cpp @@ -278,6 +278,9 @@ TEST(HTTPClient, basic_http_request) EXPECT_FALSE(result_error); EXPECT_EQ(result_status, 200); EXPECT_EQ(result_data, test_body); + + // Clean up SSL context to prevent memory leaks + HTTPClient::cleanupSSLContext(); } TEST(HTTPClient, empty_response) @@ -298,6 +301,9 @@ TEST(HTTPClient, empty_response) EXPECT_FALSE(result_error); EXPECT_EQ(result_status, 200); EXPECT_TRUE(result_data.empty()); + + // Clean up SSL context to prevent memory leaks + HTTPClient::cleanupSSLContext(); } TEST(HTTPClient, different_status_codes)