Fix for two bugs that prevents users from transfering larger amounts

[TxbitExchange]

Please make sure you checked the box ('x') before creating the pull request.

• I have tested my changes.
• I have made sure my branch is rebased. it has no merge commits nor stray commits from other people.
• [-] I have taken the base code from the dev branch, v0.11.3

(taken from master branch)

---

Description (what did you change?)

Fixed bug in RPC that limits amounts to 21M
Remove invalid money range check that limits outputs to MAX_MONEY

---

Related github issues (if there are any)

---

Notes

No consensus changes, no hard-fork required

[FreeTrade]

I wonder if this is actually a hard fork - if some nodes accept trxs with 60M+ txs and some don't, I fear there would be a chain split. Thoughts?

[TxbitExchange]

I wonder if this is actually a hard fork - if some nodes accept trxs with 60M+ txs and some don't, I fear there would be a chain split. Thoughts?

The current code accepts 60M+ transactions, so it would not be a hard fork. We were able to successfully test a 60M+ transaction and it did not cause a fork.

[FreeTrade]

That's reassuring, could you give me the txid of that transaction

[TxbitExchange]

That's reassuring, could you give me the txid of that transaction

I will send you the txid after the weekend. Our exchange had a 60M+ deposit. Block synchronization continued, validating the block that contained then 60M+ transaction. The only effect was that it could not make new transactions (sendtoaddress), or for example, request the balance (getbalance). The MAX_MONEY figure is apparently not part of the consensus checks.

[FreeTrade]

I will send you the txid after the weekend.

Thanks, I think this is probably fine, but I'd like to just verify that txid before committing the pull request.