For the first bid, the logic requires msg.value > minPrice. For subsequent bids, the check is msg.value >= previousBid increment. This creates an inconsistency: first bid must be strictly greater, while later bids allow equality.
placeBid():
Impact:
Minor UX confusion for users.
A bidder may wonder why their first bid equal to the minimum price is rejected, while equal increments later are accepted.
Standardize the check across all cases by using >=.
placeBid emits AuctionSettled even though the auction hasn’t ended, causing misleading event logs.
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.