There is no input validation for "collateral" address in the external function liquidate().
The liquidate() function in the DSCEngine.sol smart contract can be called with any address as the "collateral" address. However, doing so while creating the priceFeed object at line 344 will cause the transaction to be reverted.
AggregatorV3Interface priceFeed = AggregatorV3Interface(s_priceFeeds[token]);
When the function "liquidate()" is invoked with a non-collateral address, the transaction will be reverted.
Manual Review
Adding "isAllowedToken(collateral)" modifer to "liquidate()" function
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.