DeFiFoundry
20,000 USDC
View results
Submission Details
Severity: medium
Invalid

Auction end conditional is biased. Limits users to bid.

Summary

The auction end conditional check implemented on FjordAuction::auctionEndto whether auction has ended or not has an ambiguity. The conditional block.timestamp < auctionEndTimeis limiting users to bid by one second.

Vulnerability Details

if (block.timestamp < auctionEndTime) {
---------------------^
revert AuctionNotYetEnded();
}

Impact

Restricts users to bid more by one second. Makes auction unfair.

Tools Used

Manual Review

Recommendations

Please improve the conditional check to equality too.

- if (block.timestamp < auctionEndTime) {
+ if (block.timestamp <= auctionEndTime) {
revert AuctionNotYetEnded();
}
Updates

Lead Judging Commences

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Too generic

Support

FAQs

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