common/params.cc: unlink tmp_path only if there's an error (#32145)

1 year ago · c1edc0901e
parent 9d1b3cc773
commit c1edc0901e
1 changed files with 3 additions and 1 deletions
--- 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;
 }