common/params.cc: unlink tmp_path only if there's an error (commaai#3…

…2145)
deanlee · Apr 11, 2024 · c1edc09 · c1edc09
1 parent 9d1b3cc
commit c1edc09
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/common/params.cc b/common/params.cc
@@ -273,7 +273,9 @@ int Params::put(const char* key, const char* value, size_t value_size) {
   } while (false);
 
   close(tmp_fd);
-  ::unlink(tmp_path.c_str());
+  if (result != 0) {
+    ::unlink(tmp_path.c_str());
+  }
   return result;
 }