wallet: Exit SyncMetaData if there are no transactions to sync

laanwj · laanwj · commit b0d2ca9fb66d · 2018-05-17T22:07:32.000+02:00
Instead of crash with an assertion error, simply exit the function
`SyncMetaData` if there is no metadata to sync.

Fixes #13110.
diff --git a/src/wallet/wallet.cpp b/src/wallet/wallet.cpp
@@ -549,7 +549,9 @@ void CWallet::SyncMetaData(std::pair<TxSpends::iterator, TxSpends::iterator> ran
         }
     }
 
-    assert(copyFrom);
+    if (!copyFrom) {
+        return;
+    }
 
     // Now copy data from copyFrom to rest:
     for (TxSpends::iterator it = range.first; it != range.second; ++it)

Original file line number	Diff line number	Diff line change
`@@ -549,7 +549,9 @@ void CWallet::SyncMetaData(std::pair<TxSpends::iterator, TxSpends::iterator> ran`
`549`	`549`	`}`
`550`	`550`	`}`
`551`	`551`
`552`		`- assert(copyFrom);`
	`552`	`+ if (!copyFrom) {`
	`553`	`+ return;`
	`554`	`+ }`
`553`	`555`
`554`	`556`	`// Now copy data from copyFrom to rest:`
`555`	`557`	`for (TxSpends::iterator it = range.first; it != range.second; ++it)`