Closed victorges closed 12 months ago
What does this pull request do? Explain your changes. (required) This adds a missing checkpoint in BondingManager found during the code4rena audit.
The withdrawFees function was using autoClaimEarnings but not autoCheckpoint modifiers, so fixed that and added a test.
withdrawFees
autoClaimEarnings
autoCheckpoint
Specific updates (required)
How did you test each of these updates (required) yarn test to be deployed to devnet
yarn test
Does this pull request close any open issues? Fixes:
Checklist:
What does this pull request do? Explain your changes. (required) This adds a missing checkpoint in BondingManager found during the code4rena audit.
The
withdrawFees
function was usingautoClaimEarnings
but notautoCheckpoint
modifiers, so fixed that and added a test.Specific updates (required)
autoCheckpoint
modifier towithdrawFees
How did you test each of these updates (required)
yarn test
to be deployed to devnetDoes this pull request close any open issues? Fixes:
Checklist:
yarn test
pass