Eggstravaganza

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

No NFT Contract Initialization in Constructor

Summary

The constructor in EggVault Contract doesn’t set eggNFT, requiring a separate setEggNFT call. If the owner forgets this step, depositEgg and withdrawEgg will fail due to eggNFT being address(0).

Vulnerability Details

Impact

potential denial-of-service until initialized.

Tools Used

manual review

Recommendations

Initialize eggNFT in the constructor

constructor(address _eggNFTAddress) Ownable(msg.sender) {
require(_eggNFTAddress != address(0), "Invalid NFT address");
eggNFT = EggstravaganzaNFT(_eggNFTAddress);
}
Updates

Lead Judging Commences

m3dython Lead Judge 8 months ago
Submission Judgement Published
Invalidated
Reason: Design choice

Support

FAQs

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

Give us feedback!