hats-finance / Fenix-Finance-0x83dbe5aa378f3ce160ed084daf85f621289fb92f

0 stars 0 forks source link

VotingEscrow checkpoints is not synchronized #34

Open hats-bug-reporter[bot] opened 4 months ago

hats-bug-reporter[bot] commented 4 months ago

Github username: -- Twitter username: -- Submission hash (on-chain): 0xd676d3b0284d50f727e0b08e306ec35a46f224866b470a1de6cdabec72f25ba2 Severity: low

Description: Context: VotingEscrowUpgradeable.sol#L1215, VotingEscrowUpgradeable.sol#L1262

Description\ Delegating token ids is not synchronizing correctly timestamp variable in the checkpoint, by leaving it not updated the functions getPastVotesIndex and _findWhatCheckpointToWrite could return the incorrect index.

Recommendation: It is recommended to update the fromBlock variable of checkpoint in functions _moveTokenDelegates and _moveAllDelegates

BohdanHrytsak commented 4 months ago

Thank you for the submission.

Duplicate: #30