Fix static analysis issues 93/290293/3
authorSangyoon Jang <jeremy.jang@samsung.com>
Thu, 23 Mar 2023 00:13:41 +0000 (09:13 +0900)
committerSangyoon Jang <jeremy.jang@samsung.com>
Thu, 23 Mar 2023 00:24:19 +0000 (09:24 +0900)
Change-Id: Iba46bb2ddb6359bf54f0a15e1c4e922b87a3c421
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
src/wgt/utils/extension_config_parser.cc

index 93d0abf..7583c46 100644 (file)
@@ -58,6 +58,10 @@ std::string ExtensionConfigParser::GetNodeDir(
     if (xmlStrEqual(prop->name, kDirAttributeKey)) {
       char* prop_value = reinterpret_cast<char*>(xmlNodeListGetString(
           node->doc, prop->children, 1));
+      if (!prop_value) {
+        LOG(ERROR) << "Failed to get value string";
+        return {};
+      }
       dir = prop_value;
       xmlFree(prop_value);
       break;
@@ -116,6 +120,10 @@ std::unique_ptr<parser::DictionaryValue>
   xmlAttr* prop = nullptr;
   for (prop = root->properties; prop; prop = prop->next) {
     xmlChar* value_ptr = xmlNodeListGetString(root->doc, prop->children, 1);
+    if (!value_ptr) {
+      LOG(ERROR) << "Failed to get value string";
+      continue;
+    }
     std::string prop_value(reinterpret_cast<char*>(value_ptr));
     xmlFree(value_ptr);
     if (IsPropSupportDir(root, prop))