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

Wallet Crashes to desktop when voting on expired poll. #1979

Closed
ShmoogleOsukami opened this issue Dec 3, 2020 · 1 comment · Fixed by #1980
Closed

Wallet Crashes to desktop when voting on expired poll. #1979

ShmoogleOsukami opened this issue Dec 3, 2020 · 1 comment · Fixed by #1980
Assignees
Labels
Milestone

Comments

@ShmoogleOsukami
Copy link
Member

ShmoogleOsukami commented Dec 3, 2020

This was reported on CCT forum by Typv0deister and I tested it and was able to replicate the problem.

Image of error: https://i.gyazo.com/2381dd88a6a07c3c0f4012aa39b61041.png

Forum post: https://cryptocurrencytalk.com/topic/169561-gridcoin-client-crash-on-voting/

replicated on Wallet version 5.1.0.0

@ShmoogleOsukami ShmoogleOsukami changed the title Wallet Crashes to desktop when voting on expired vote. Wallet Crashes to desktop when voting on expired poll. Dec 3, 2020
@jamescowens
Copy link
Member

jamescowens commented Dec 3, 2020

This is confirmed. The VoteBuilder checks for an expired poll and throws a VotingError, but currently all of these are handled as std::runtime_errors. There is a VoteBuilder assignment which is not in a try block and allows the exception to go all the way up to the main application level and stops the execution of the wallet. This is not necessary and ungraceful. I am trying out putting in the missing try-catch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants