Eggstravaganza

First Flight #37
Beginner FriendlySolidity
100 EXP
View results
Submission Details
Severity: low
Invalid

Centralization Risk

Summary

Centralization Risk

Vulnerability Details

The owner of EggHuntGame has privileged rights to perform admin tasks and need to be trusted to not perform malicious updates.

/// @notice Allows the owner to adjust the egg-finding chance.
@> function setEggFindThreshold(uint256 newThreshold) external onlyOwner {
require(newThreshold <= 100, "Threshold must be <= 100");
eggFindThreshold = newThreshold;
}

Impact

The owner can for example increase eggFindThreshold to a higher percentage making it virtually impossible/less possible for someone to find a random number below that percentage to mint an NFT.

Tools Used

  • Manual review

  • Aderyn static analyzer

Recommendations

Probably make the eggFindThreshold a constant.

Updates

Lead Judging Commences

m3dython Lead Judge about 2 months ago
Submission Judgement Published
Invalidated
Reason: Design choice
Assigned finding tags:

Trusted Owner

Owner is trusted and is not expected to interact in ways that would compromise security

Support

FAQs

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