-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
wallet: fix pre-rct cold wallet signing not splitting change #1261
wallet: fix pre-rct cold wallet signing not splitting change #1261
Conversation
Actually, wait a bit, I'll replace with some more stuff (and bump version) |
Hate to be this guy, but the past participle of 'to split' is 'split' rather than 'splitted'. English makes no sense... |
Re-creating the transaction on the cold wallet was not splitting the change, causing the transaction to be rejected by the network. This worked on testnet since amounts do not have to be split. Also add selected_transfers, which can now be saved since they're size_t rather than iterators. This allows the view wallet to properly set the sent outputs as spent and update balance. Bump transfer file version numbers to match.
724960c
to
31abac4
Compare
done |
split, splat, splut ? |
Hmm, that's used in many other places, so it'll have to stay that way. |
Not as trustworthy as this is in the view wallet, the one that's considered compromised.
39267ef
to
069d688
Compare
Small update, I'd forgot to add a file to the commit (the one that actually logs the tx). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed
Re-creating the transaction on the cold wallet was not splitting
the change, causing the transaction to be rejected by the network.
This worked on testnet since amounts do not have to be split.