init, validation: Improve handling if VerifyDB() fails due to insufficient dbcache

The rpc command verifychain now fails if the dbcache was not sufficient
to complete the verification at the specified level and depth.

In the same situation, the VerifyDB check during Init will now fail (and lead to
an early shutdown) if the user has explicitly specified -checkblocks or
-checklevel but the check couldn't be executed because of the limited
cache. If the user didn't change any of the two and is using the defaults, log a warning
but don't prevent the node from starting up.

1 files changed, 3 insertions, 0 deletions

diff --git a/src/validation.cpp b/src/validation.cpp
index 5a6db2674d..4da52b7d8b 100644
--- a/src/validation.cpp
+++ b/src/validation.cpp
@@ -4185,6 +4185,9 @@ VerifyDBResult CVerifyDB::VerifyDB(
 
     LogPrintf("Verification: No coin database inconsistencies in last %i blocks (%i transactions)\n", block_count, nGoodTransactions);
 
+    if (skipped_l3_checks) {
+        return VerifyDBResult::SKIPPED_L3_CHECKS;
+    }
     return VerifyDBResult::SUCCESS;
 }