20,000 USDC
View results
Submission Details
Severity: medium

not min duration auctionLength can lead to lender getting borrowers collateral

Summary

user validated input for auction length only checks the value not being 0, but, a for example 1s auctionLength can lead to a "instant" liquidation of the borrower losing its collateral, also resulting in an inability for anyone to participate in the dutch auction.

Vulnerability Details

File: src/Lender.sol
´´´
// the only check to auctionLength
136: p.auctionLength == 0 ||

471: if (block.timestamp > loan.auctionStartTimestamp + loan.auctionLength)
472: revert AuctionEnded();

## Impact
Borrower can lose its collateral in any moment the lender decides to.
## Tools Used
Manual Review
## Recommendations
Set a minimum duration for auction length to let the borrower react in a considerable amount of time

Support

FAQs

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