From b7be35bec10de7cc2963e13f783f23792265b8c9 Mon Sep 17 00:00:00 2001
From: xianju6x <xianjunx.zheng@intel.com>
Date: Wed, 15 Feb 2023 13:49:32 +0800
Subject: [PATCH] fix the issues scanned by Coverity

---
 src/guest/config_parser.cc | 4 +---
 src/guest/vm_flash.cc      | 7 +++----
 src/utils/utils.cc         | 6 ++++--
 3 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/src/guest/config_parser.cc b/src/guest/config_parser.cc
index 6bac5852..b8f6f2e7 100644
--- a/src/guest/config_parser.cc
+++ b/src/guest/config_parser.cc
@@ -53,9 +53,7 @@ bool CivConfig::SanitizeOpts(void) {
         if (group != kConfigMap.end()) {
             for (auto& subsec : section.second) {
                 auto key = std::find(group->second.begin(), group->second.end(), subsec.first);
-                if (key != group->second.end()) {
-                    continue;
-                } else {
+                if (key == group->second.end()) {
                     LOG(error) << "Invalid key: " << section.first << "." << subsec.first << "\n";
                     return false;
                 }
diff --git a/src/guest/vm_flash.cc b/src/guest/vm_flash.cc
index 6b64018c..83e2647f 100644
--- a/src/guest/vm_flash.cc
+++ b/src/guest/vm_flash.cc
@@ -269,15 +269,14 @@ bool VmFlasher::FlashGuest(std::string path) {
 
     if (!cfg_.ReadConfigFile(p.string())) {
         LOG(error) << "Failed to read config file";
-        return -1;
+        return false;
     }
 
     std::string emul_type = cfg_.GetValue(kGroupEmul, kEmulType);
-    if (emul_type.compare(kEmulTypeQemu) == 0) {
-        return FlashWithQemu();
-    } else {
+    if ((emul_type.compare(kEmulTypeQemu) == 0) || emul_type.empty()) {
         return FlashWithQemu();
     }
+    return false;
 }
 
 }  // namespace vm_manager
diff --git a/src/utils/utils.cc b/src/utils/utils.cc
index da61b22a..5d24d4ea 100644
--- a/src/utils/utils.cc
+++ b/src/utils/utils.cc
@@ -48,7 +48,7 @@ const char *GetConfigPath(void) {
     }
 
     if (sudo_uid) {
-        euid = atoi(sudo_gid);
+        euid = atoi(sudo_uid);
         setresuid(ruid, euid, suid);
     }
 
@@ -106,8 +106,10 @@ int Daemonize(void) {
     close(STDERR_FILENO);
 
     int fd = open("/dev/null", O_RDWR);
-    if (fd != STDIN_FILENO)
+    if (fd != STDIN_FILENO) {
+        close(fd);
         return -1;
+    }
     if (dup2(STDIN_FILENO, STDOUT_FILENO) != STDOUT_FILENO)
         return -1;
     if (dup2(STDIN_FILENO, STDERR_FILENO) != STDERR_FILENO)