ci: fix ci ubasan failure
remove an unused std::shared_ptr<Subscription>& from
void Subscription::resHandler(...)
Since it was just a reference, it was non-owning anyways and removing it
should not have any impact to the lifetime of the managed object.
Tested: Inspection only.
Change-Id: Iab57e456d5a7ae32305e1a38ddcc37c0f0156ed4
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>
diff --git a/redfish-core/include/subscription.hpp b/redfish-core/include/subscription.hpp
index 2f93f74..e7f1715 100644
--- a/redfish-core/include/subscription.hpp
+++ b/redfish-core/include/subscription.hpp
@@ -75,8 +75,7 @@
~Subscription() = default;
// callback for subscription sendData
- void resHandler(const std::shared_ptr<Subscription>& /*unused*/,
- const crow::Response& res);
+ void resHandler(const crow::Response& res);
void sendHeartbeatEvent();
void scheduleNextHeartbeatEvent();
diff --git a/redfish-core/src/subscription.cpp b/redfish-core/src/subscription.cpp
index 00c9036..5f4bf0f 100644
--- a/redfish-core/src/subscription.cpp
+++ b/redfish-core/src/subscription.cpp
@@ -76,8 +76,7 @@
{}
// callback for subscription sendData
-void Subscription::resHandler(const std::shared_ptr<Subscription>& /*unused*/,
- const crow::Response& res)
+void Subscription::resHandler(const crow::Response& res)
{
BMCWEB_LOG_DEBUG("Response handled with return code: {}", res.resultInt());
@@ -201,8 +200,7 @@
static_cast<ensuressl::VerifyCertificate>(
userSub->verifyCertificate),
httpHeadersCopy, boost::beast::http::verb::post,
- std::bind_front(&Subscription::resHandler, this,
- shared_from_this()));
+ std::bind_front(&Subscription::resHandler, this));
return true;
}