From fa54e38cd6b0e73da6eb06854006f1983e712ec3 Mon Sep 17 00:00:00 2001
From: Pawel Wasowski
Date: Tue, 18 May 2021 16:11:12 +0200
Subject: [PATCH] [ML][common] Fix a Coverity issue
Coverity issue numver: 1229783
This commit fixes a potential memory leak.
[Verification] The code compiles (it's hard to provoke the code to enter
the problematic branch, so the new code wasn't executed)
Change-Id: Id560ffdffc18ff6bbf2bc987e8e0ad6853b2cdb6
---
src/ml/ml_tensors_info_manager.cc | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/src/ml/ml_tensors_info_manager.cc b/src/ml/ml_tensors_info_manager.cc
index 46623df0..431b20df 100644
--- a/src/ml/ml_tensors_info_manager.cc
+++ b/src/ml/ml_tensors_info_manager.cc
@@ -105,6 +105,10 @@ std::shared_ptr TensorsInfo::CreateClone(int cloneId) {
ret = ml_tensors_info_clone(clone_h, this->Handle());
if (ML_ERROR_NONE != ret) {
LoggerE("ml_tensors_info_clone failed: %d (%s)", ret, get_error_message(ret));
+ ret = ml_tensors_info_destroy(clone_h);
+ if (ML_ERROR_NONE != ret) {
+ LoggerE("ml_tensors_info_destroy failed: %d (%s)", ret, get_error_message(ret));
+ }
return nullptr;
}
auto t = std::make_shared(clone_h, cloneId);
--
2.34.1