Beginner FriendlyFoundryNFT
100 EXP
View results
Submission Details
Severity: low
Invalid

PuppyRaffle::changeFeeAddress allows to change the owner directly

Summary

PuppyRaffle.changeFeeAddress() changes the owner in one step and doesn't validate the acceptance capability of the new owner.

Vulnerability Details

  • the new owner can be zero address or a contract that couldn't handle the calls of PuppyRaffle calls.

Impact

  • severity : low

  • likelihood : low

Tools Used

  • manual verification

Recommendations

  • Inherit the Ownable2Step of openZeppelin instead of Ownable.

Updates

Lead Judging Commences

Hamiltonite Lead Judge almost 2 years ago
Submission Judgement Published
Invalidated
Reason: User experience and design improvement

Support

FAQs

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