[wallet] Close DB on error.

Ref: bitcoin/bitcoin#11017
gridcoin-community · Jul 12, 2021 · bca2f93 · bca2f93
1 parent 57bf77b
commit bca2f93
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/src/wallet/db.cpp b/src/wallet/db.cpp
@@ -101,8 +101,10 @@ bool CDBEnv::Open(fs::path pathEnv_)
                      DB_RECOVER    |
                      nEnvFlags,
                      S_IRUSR | S_IWUSR);
-    if (ret != 0)
+    if (ret != 0) {
+        dbenv.close(0);
         return error("CDB() : error %s (%d) opening database environment", DbEnv::strerror(ret), ret);
+    }
 
     fDbEnvInit = true;
     fMockDb = false;
@@ -376,9 +378,9 @@ bool CDB::Rewrite(const string& strFile, const char* pszSkip)
                                             DB_BTREE,           // Database type
                                             DB_CREATE,          // Flags
                                             0);
-                    if (ret > 0)
-                    {
+                    if (ret > 0) {
                         LogPrintf("Cannot create database file %s", strFileRes);
+                        pdbCopy->close(0);
                         fSuccess = false;
                     }
 
@@ -415,14 +417,15 @@ bool CDB::Rewrite(const string& strFile, const char* pszSkip)
                             if (ret2 > 0)
                                 fSuccess = false;
                         }
-                    if (fSuccess)
-                    {
+                    if (fSuccess) {
                         db.Close();
                         bitdb.CloseDb(strFile);
                         if (pdbCopy->close(0))
                             fSuccess = false;
-                        delete pdbCopy;
+                    } else {
+                        pdbCopy->close(0);
                     }
+                    delete pdbCopy;
                 }
                 if (fSuccess)
                 {