Submission Details
Severity:
Unused return value from mintEgg()
Description:
The searchForEgg() function calls eggNFT.mintEgg() but ignores its return value:
// @audit-info return not used
eggNFT.mintEgg(msg.sender, eggCounter);
Impact:
While the function has require() checks, ignoring the return value means:
Potential missed error handling if mint fails
Could mask reentrancy issues
Poor code quality practice
Recommended Mitigation:
Either:
Remove the return value from mintEgg() if not needed
Or properly handle the return value in searchForEgg()
Likelihood and Impact
Impact: Low
Likelihood: Low
Severity: Low
Updates
Lead Judging Commences
m3dython 8 months ago
Submission Judgement Published Reason: Non-acceptable severity
Assigned finding tags:
Unused return value
Returns a boolean value that isn't utilized by its caller
Rewards breakdown
nSLOC
129This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.