Weather Witness

First Flight #40
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Submission Details
Impact: high
Likelihood: high
Invalid

Missing Reentrancy Guard

Description: 'requestMintWeatherNFT' makes external calls ('safeTransferFrom' for LINK, Chainlink request) without 'nonReentrant'.

Impact: A malicious LINK token could reenter and manipulate contract state (e.g., call 'requestMintWeatherNFT' recursively before 's_currentMintPrice' updates).

Recommended Mitigation: Inherit OpenZeppelin's ReentrancyGuard and add nonReentrant:

function requestMintWeatherNFT(...) external payable nonReentrant { ... }
Updates

Appeal created

bube Lead Judge 4 months ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement

Support

FAQs

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