From: Piotr Kosko/Tizen API (PLT) /SRPOL/Engineer/Samsung Electronics <p.kosko@samsung.com>
Date: Fri, 19 Mar 2021 11:20:55 +0000 (+0100)
Subject: [Common][ML] Fixing SVACE issues and code formatting
X-Git-Tag: submit/tizen/20210326.074032~1^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0dc57775dde46610adc7774c08e86d93f89a21d3;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

[Common][ML] Fixing SVACE issues and code formatting

[SVACE] 1222586, 1222593

[Verification] Code compiles without errors.

Change-Id: Ibf34be9edb25f1b4e61e8480c9b6ad42262d42a4
---

diff --git a/src/common/converter.h b/src/common/converter.h
index 5ba7a6b5..db0955d1 100644
--- a/src/common/converter.h
+++ b/src/common/converter.h
@@ -59,8 +59,8 @@ const T &FromJson(const picojson::object &in, const char *name, Names... names)
   return FromJson<T>(v.get<picojson::object>(), names...);
 }
 
-void encode_binary_in_string(const std::vector<std::uint8_t>& buf, std::string& str);
-void decode_binary_from_string(const std::string& str, std::vector<std::uint8_t>& buf);
+void encode_binary_in_string(const std::vector<std::uint8_t> &buf, std::string &str);
+void decode_binary_from_string(const std::string &str, std::vector<std::uint8_t> &buf);
 
 }  // common
 
diff --git a/src/common/picojson.h b/src/common/picojson.h
index 54a40a74..83b6e93d 100644
--- a/src/common/picojson.h
+++ b/src/common/picojson.h
@@ -916,7 +916,7 @@ inline bool _parse(Context &ctx, input<Iter> &in) {
           }
         }
 #endif
-        locale_t loc = newlocale(LC_NUMERIC_MASK, "C", (locale_t) 0);
+        locale_t loc = newlocale(LC_NUMERIC_MASK, "C", (locale_t)0);
         f = strtod_l(num_str.c_str(), &endp, loc);
         freelocale(loc);
         if (endp == num_str.c_str() + num_str.size()) {
diff --git a/src/common/tools.cc b/src/common/tools.cc
index 21f28b9e..9e9368e9 100644
--- a/src/common/tools.cc
+++ b/src/common/tools.cc
@@ -165,6 +165,10 @@ class AccessControlImpl {
     ScopeLogger("Privilege access checked using Cynara.");
 
     char* smack_label = nullptr;
+    SCOPE_EXIT {
+      free(smack_label);
+    };
+
     char path[1024] = {
         0,
     };
@@ -179,8 +183,6 @@ class AccessControlImpl {
 
       uid_ = std::to_string(uid);
       smack_label_ = smack_label;
-
-      free(smack_label);
     } else {
       LoggerE("Failed to get smack label");
       return;
diff --git a/src/ml/ml_pipeline_nodeinfo.cc b/src/ml/ml_pipeline_nodeinfo.cc
index d01f997b..3132d93b 100644
--- a/src/ml/ml_pipeline_nodeinfo.cc
+++ b/src/ml/ml_pipeline_nodeinfo.cc
@@ -57,7 +57,7 @@ PlatformResult NodeInfo::CreateNodeInfo(ml_pipeline_h pipeline, const std::strin
   return PlatformResult{};
 }
 
-NodeInfo::NodeInfo(const std::string& name) : name_{name} {
+NodeInfo::NodeInfo(const std::string& name) : name_{name}, node_info_{nullptr} {
   ScopeLogger("name: [%s] ", name.c_str());
 }