Fix covertity issue

It fixes coverity issues:
- 87144
- 87145
- 100780

Signed-off-by: Jiho Chu <jiho.chu@samsung.com>

diff --git a/tests/unittests/ne_core_npu_test.cc b/tests/unittests/ne_core_npu_test.cc
index 7600c04752a583a30e55f6e3f85334d95bed7b3a..5fd2451de3cf673a9011fc838184c5414b8cdb0b 100644 (file)
--- a/tests/unittests/ne_core_npu_test.cc
+++ b/tests/unittests/ne_core_npu_test.cc
@@ -357,7 +357,7 @@ TEST_F (NpuTest, run_inference_triv2) {
 TEST_F (NpuTest, run_inference_async0_triv2) {
   const size_t size = 4096;
   std::future<int> f_ret;
-  int seg_dmabuf, model_dmabuf;
+  int seg_dmabuf = -1, model_dmabuf = -1;
 
   /** register model data */
   model_dmabuf = api->alloc (size);

diff --git a/utils/trinity_test/stress_test.cc b/utils/trinity_test/stress_test.cc
index 91ff3e7613a6742ea2a3ffaa38eae42890729c5c..d613cd6519e9c7df53b49eb343ef1b10d2ee882c 100644 (file)
--- a/utils/trinity_test/stress_test.cc
+++ b/utils/trinity_test/stress_test.cc
@@ -97,13 +97,15 @@ StressTester::addRequest () {
 
   if (input_buffer == nullptr || output_buffer == nullptr) {
     std::cerr << "Failed to create buffers" << std::endl;
-    return -EINVAL;
+    status = -EINVAL;
+    goto delete_buffers;
   }
 
   status = allocNPU_genericBuffers (dev_, output_buffer);
   if (status < 0) {
     std::cerr << "Failed to alloc output buffers : " << status << std::endl;
-    return -EINVAL;
+    status = -EINVAL;
+    goto delete_buffers;
   }
 
   status = createNPU_request (dev_, model_id_, &req_id);
@@ -133,6 +135,11 @@ remove_request:
   removeNPU_request (dev_, req_id);
 clean_output:
   cleanNPU_genericBuffers (dev_, output_buffer);
+delete_buffers:
+  if (input_buffer)
+    delete input_buffer;
+  if (output_buffer)
+    delete output_buffer;
 
   return status;
 }