diff options
author | Gavin Andresen <gavinandresen@gmail.com> | 2011-01-20 13:10:01 -0500 |
---|---|---|
committer | Gavin Andresen <gavinandresen@gmail.com> | 2011-01-20 13:10:01 -0500 |
commit | d9574c2f14028297ad5121695a0c10e517bf638e (patch) | |
tree | 6f5adb8c454cf303c63ba7363e675501ac4b9b51 /rpc.cpp | |
parent | 1d23c7436164faa0b1822acf5ce9507bf6ff257a (diff) | |
download | bitcoin-d9574c2f14028297ad5121695a0c10e517bf638e.tar.xz |
Reconcile getbalance and listaccounts 0 in the shared-wallet case
If you copied your wallet and used it on two different machines, the balance reported by getbalance and the sum(listaccounts) could disagree, because you might receive payments for an address that is in your wallet but not your address book. Now all such transactions are credited to the default empty-string account.
Diffstat (limited to 'rpc.cpp')
-rw-r--r-- | rpc.cpp | 8 |
1 files changed, 8 insertions, 0 deletions
@@ -626,6 +626,7 @@ Value getbalance(const Array& params, bool fHelp) // (GetBalance() sums up all unspent TxOuts) // getbalance and getbalance '*' should always return the same number. int64 nBalance = 0; + vector<string> vAccounts; for (map<uint256, CWalletTx>::iterator it = mapWallet.begin(); it != mapWallet.end(); ++it) { const CWalletTx& wtx = (*it).second; @@ -636,12 +637,17 @@ Value getbalance(const Array& params, bool fHelp) list<pair<string, int64> > listSent; wtx.GetAmounts(allGenerated, listReceived, listSent, allFee, strSentAccount); foreach(const PAIRTYPE(string,int64)& r, listReceived) + { nBalance += r.second; + if (!count(vAccounts.begin(), vAccounts.end(), r.first)) + vAccounts.push_back(r.first); + } foreach(const PAIRTYPE(string,int64)& r, listSent) nBalance -= r.second; nBalance -= allFee; nBalance += allGenerated; } + printf("Found %d accounts\n", vAccounts.size()); return (double)nBalance / (double)COIN; } @@ -1080,6 +1086,8 @@ Value listaccounts(const Array& params, bool fHelp) foreach(const PAIRTYPE(string, int64)& r, listReceived) if (mapAddressBook.count(r.first)) mapAccountBalances[mapAddressBook[r.first]] += r.second; + else + mapAccountBalances[""] += r.second; } } } |