wallet2: don't write a .new file when not needed

This commit is contained in:
moneromooo-monero 2017-12-07 12:07:18 +00:00
parent 8512a83572
commit 3f24e90149
No known key found for this signature in database
GPG key ID: 686F07454D6CEFC3

View file

@ -2960,14 +2960,6 @@ void wallet2::store_to(const std::string &path, const std::string &password)
const std::string old_keys_file = m_keys_file;
const std::string old_address_file = m_wallet_file + ".address.txt";
// save to new file
std::ofstream ostr;
ostr.open(new_file, std::ios_base::binary | std::ios_base::out | std::ios_base::trunc);
binary_archive<true> oar(ostr);
bool success = ::serialization::serialize(oar, cache_file_data);
ostr.close();
THROW_WALLET_EXCEPTION_IF(!success || !ostr.good(), error::file_save_error, new_file);
// save keys to the new file
// if we here, main wallet file is saved and we only need to save keys and address files
if (!same_file) {
@ -2994,6 +2986,14 @@ void wallet2::store_to(const std::string &path, const std::string &password)
LOG_ERROR("error removing file: " << old_address_file);
}
} else {
// save to new file
std::ofstream ostr;
ostr.open(new_file, std::ios_base::binary | std::ios_base::out | std::ios_base::trunc);
binary_archive<true> oar(ostr);
bool success = ::serialization::serialize(oar, cache_file_data);
ostr.close();
THROW_WALLET_EXCEPTION_IF(!success || !ostr.good(), error::file_save_error, new_file);
// here we have "*.new" file, we need to rename it to be without ".new"
std::error_code e = tools::replace_file(new_file, m_wallet_file);
THROW_WALLET_EXCEPTION_IF(e, error::file_save_error, m_wallet_file, e);