Mirrors
/
monero
mirror of https://github.com/monero-project/monero.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

LMDB: protection against UB, by not calling virtual methods from destructor - v0.17

This commit is contained in:
mj-xmr 2021-08-16 08:08:26 +02:00
parent 2222bea92f
commit cf65e5b41b
No known key found for this signature in database
GPG Key ID: C101BF94093451E0
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/blockchain_db/lmdb/db_lmdb.cpp
View File

@ -1268,11 +1268,11 @@ BlockchainLMDB::~BlockchainLMDB()
// batch transaction shouldn't be active at this point. If it is, consider it aborted. // batch transaction shouldn't be active at this point. If it is, consider it aborted.
if (m_batch_active) if (m_batch_active)
{ {
try { batch_abort(); } try { BlockchainLMDB::batch_abort(); }
catch (...) { /* ignore */ } catch (...) { /* ignore */ }
} }
if (m_open) if (m_open)
close(); BlockchainLMDB::close();
} }
BlockchainLMDB::BlockchainLMDB(bool batch_transactions): BlockchainDB() BlockchainLMDB::BlockchainLMDB(bool batch_transactions): BlockchainDB()
@ -1569,9 +1569,9 @@ void BlockchainLMDB::close()
if (m_batch_active) if (m_batch_active)
{ {
LOG_PRINT_L3("close() first calling batch_abort() due to active batch transaction"); LOG_PRINT_L3("close() first calling batch_abort() due to active batch transaction");
batch_abort(); BlockchainLMDB::batch_abort();
} }
this->sync(); BlockchainLMDB::sync();
m_tinfo.reset(); m_tinfo.reset();
// FIXME: not yet thread safe!!! Use with care. // FIXME: not yet thread safe!!! Use with care.
@ -1584,7 +1584,7 @@ void BlockchainLMDB::sync()
LOG_PRINT_L3("BlockchainLMDB::" << __func__); LOG_PRINT_L3("BlockchainLMDB::" << __func__);
check_open(); check_open();
if (is_read_only()) if (BlockchainLMDB::is_read_only())
return; return;
// Does nothing unless LMDB environment was opened with MDB_NOSYNC or in part // Does nothing unless LMDB environment was opened with MDB_NOSYNC or in part