DittoETH

Ditto
DeFiFoundryOracle
55,000 USDC
View results
Submission Details
Severity: low
Invalid

Incorrect comment on `_canLiquidate()` natspec

Summary

It's not by 'liquidated by shorter', but, 'liquidated by flagger', shorter cannot liquidate himself.

Vulnerability Details

Incorrect comment natspect on _canLiquidate, in contracts/facets/MarginCallPrimaryFacet.sol.

Impact

Very low, but can mislead to non tech users reading the liquidate code docs.

Tools Used

Manual review.

Recommendations

Correction:

- * @dev ...After that, the flagger has 2 hours to liquidate the shorter. If short is not liquidated by shorter within that time, ANYBODY can then liquidate...`
+ * @dev ...After that, the flagger has 2 hours to liquidate the shorter. If short is not liquidated by flagger within that time, ANYBODY can then liquidate...`
Updates

Lead Judging Commences

0xnevi Lead Judge
over 1 year ago
0xnevi Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Other

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.