When migrating to a new staking contract, you should only allow transfers to be done when the current one is paused.
With migrateWithdraw
lacking the modifier whenPaused
, the assigned onlyMigrator
can start transferring funds of users even when the current contract is still live.
This can cause confusion of users to the system, or the executor could forget to pause and a endless loop of people staking still happening and the executor calling migrateWithdraw
.
Foundry
Add the whenPaused
modifier to migrateWithdraw
-
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.