Clarify *(--.end()) iterator semantics in CWallet::TopUpKeyPool

7 years ago · d40a72ccbb
parent 28301b9780
commit d40a72ccbb
1 changed files with 2 additions and 2 deletions
--- a/src/wallet/wallet.cpp
+++ b/src/wallet/wallet.cpp
@ -3141,10 +3141,10 @@ bool CWallet::TopUpKeyPool(unsigned int kpSize)
            }

            if (!setInternalKeyPool.empty()) {
-                nEnd = *(--setInternalKeyPool.end()) + 1;
+                nEnd = *(setInternalKeyPool.rbegin()) + 1;
            }
            if (!setExternalKeyPool.empty()) {
-                nEnd = std::max(nEnd, *(--setExternalKeyPool.end()) + 1);
+                nEnd = std::max(nEnd, *(setExternalKeyPool.rbegin()) + 1);
            }

            if (!walletdb.WritePool(nEnd, CKeyPool(GenerateNewKey(internal), internal)))