search

hats-finance / Fenix-Finance-0x83dbe5aa378f3ce160ed084daf85f621289fb92f

0 stars 0 forks source link

Missing nonReentrant modifier on a state changing checkpoint function #31

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): 0x243bf1f52ff9d94b3717d6277db4422264c51c5f64b4f7c2252ae483e3d4544e Severity: low

Description: Context: VotingEscrowUpgradeable.sol#L722-L724

Description\ The checkpoint() function will call the internal _checkpoint() function which ultimately fills the point history and potentially updates the epoch state variable.

Attack Scenario\ Describe how the vulnerability can be exploited.

Recommendation: Add the nonReentrant modifier to the external checkpoint() function.

BohdanHrytsak commented 4 months ago

Thank you for the submission.

Refers to inherited code from Thena & Chronos, OOS