From f6cc9012db2ec2ff500d5e44c6007e5a62364673 Mon Sep 17 00:00:00 2001 From: Richard Fuchs Date: Wed, 12 Mar 2025 10:50:33 -0400 Subject: [PATCH] MT#59962 click2dial: use smart pointer Elimiates a possible leak. Change-Id: Ic1d0fe61075765f77dbc03fd40f934e5828e027e Warned-by: Coverity --- apps/click2dial/Click2Dial.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/apps/click2dial/Click2Dial.cpp b/apps/click2dial/Click2Dial.cpp index 93b3168c..ad5d6821 100644 --- a/apps/click2dial/Click2Dial.cpp +++ b/apps/click2dial/Click2Dial.cpp @@ -104,7 +104,6 @@ string Click2DialFactory::getAnnounceFile(const AmSipRequest& req) AmSession* Click2DialFactory::onInvite(const AmSipRequest& req, const string& app_name, AmArg& session_params) { - UACAuthCred* cred = NULL; string callee_uri, a_realm, a_user, a_pwd; if(session_params.size() != 4) { @@ -135,7 +134,7 @@ AmSession* Click2DialFactory::onInvite(const AmSipRequest& req, const string& ap return NULL; } - cred = new UACAuthCred(a_realm, a_user, a_pwd); + auto cred = std::make_unique(a_realm, a_user, a_pwd); if(cred == NULL) { ERROR("Failed to create authentication handle\n"); return NULL; @@ -149,7 +148,7 @@ AmSession* Click2DialFactory::onInvite(const AmSipRequest& req, const string& ap return NULL; } - AmSession* s = new C2DCallerDialog(req, getAnnounceFile(req), callee_uri, cred); + AmSession* s = new C2DCallerDialog(req, getAnnounceFile(req), callee_uri, cred.release()); if(s == NULL) { ERROR("Failed to create a click2dial dialog"); return NULL;