15,000 USDC
View results
Submission Details
Severity: medium
Valid

In liquidate() _revertIfHealthFactorIsBroken(msg.sender) can be simply bypassed.

Summary

Last line of liquidate(), _revertIfHealthFactorIsBroken(msg.sender); can simply be bypassed.

Vulnerability Details

Anyone can simply make an account and bypass _revertIfHealthFactorIsBroken(msg.sender);. Since _revertIfHealthFactorIsBroken(msg.sender); does not revert when it does not have DSC minted.

Tools Used

vscode

Recommendations

make only user who provided collateral can liquidate other user.

Support

FAQs

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