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
fix: [L02]: Duplicate function execution in commitAndEmitEncryptedVote #4069
Conversation
Signed-off-by: chrismaree <christopher.maree@gmail.com>
Signed-off-by: chrismaree <christopher.maree@gmail.com>
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.
LGTM once extra changes are removed!
function getStartingIndexForStaker() internal virtual returns (uint64) { | ||
return 0; | ||
} | ||
function getStartingIndexForStaker() internal view virtual returns (uint64) {} |
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.
nit: I think changes from #4068 were accidentally included. We should probably drop this commit.
@@ -611,8 +611,6 @@ contract VotingV2 is | |||
bytes32 hash, | |||
bytes memory encryptedVote | |||
) public override { | |||
commitVote(identifier, time, "", hash); |
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.
Wow, this is a great catch!
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.
Nice catch. LGTM
…ithub.com:UMAprotocol/protocol
Motivation
OZ identified the following issue:
Fix presented in this PR:
Remove the duplicate call to
commitVote
.Testing
Check a box to describe how you tested these changes and list the steps for reviewers to test.