]> git.ipfire.org Git - thirdparty/pdns.git/commitdiff
rec: Fix a warning from Coverity (CID 1522877) 13435/head
authorRemi Gacogne <remi.gacogne@powerdns.com>
Tue, 31 Oct 2023 09:45:23 +0000 (10:45 +0100)
committerRemi Gacogne <remi.gacogne@powerdns.com>
Tue, 31 Oct 2023 09:45:23 +0000 (10:45 +0100)
pdns/recursordist/settings/cxxsupport.cc

index 511305930a28bbc990203473a323109751dc57a7..efb328877ecf1b0b5f2f861fa392029f207123c1 100644 (file)
@@ -200,7 +200,10 @@ static void possiblyConvertACLFile(const string& includeDir, const string& apiDi
   if (rename(tmpfilename.c_str(), yamlfilename.c_str()) != 0) {
     int err = errno;
     log->error(Logr::Error, err, "Rename failed", "file", Logging::Loggable(tmpfilename), "to", Logging::Loggable(yamlfilename));
-    rename((path + ".converted").c_str(), path.c_str());
+    if (rename((path + ".converted").c_str(), path.c_str()) != 0) {
+      err = errno;
+      log->error(Logr::Error, err, "Rename failed", "file", Logging::Loggable(path + ".converted"), "to", Logging::Loggable(path));
+    }
     throw runtime_error("YAML Conversion");
   }
   log->info(Logr::Notice, "Converted to YAML", "file", Logging::Loggable(path), "to", Logging::Loggable(yamlfilename));