Skip to content
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

Fixed bug in sigma spend tx creation #594

Merged
merged 1 commit into from Aug 8, 2019

Conversation

@psolstice
Copy link
Contributor

commented Aug 8, 2019

PR intention

Fixes rare bug in creating sigma spend transaction

Code changes brief

CSigmaState::GetCoinSetForSpend() added items to its output without clearing it first. This PR fixes this behavior.

@reubenyap reubenyap added the bug label Aug 8, 2019

@reubenyap reubenyap added this to the v0.13.8.4 milestone Aug 8, 2019

@reubenyap reubenyap added this to Needs review in Zcoin Core via automation Aug 8, 2019

Zcoin Core automation moved this from Needs review to Reviewer approved Aug 8, 2019

@psolstice psolstice merged commit fd9d321 into master Aug 8, 2019

2 of 4 checks passed

LGTM analysis: JavaScript No code changes detected
Details
LGTM analysis: Python No code changes detected
Details
continuous-integration/jenkins/branch This commit looks good
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details

Zcoin Core automation moved this from Reviewer approved to Done Aug 8, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
4 participants
You can’t perform that action at this time.