Bid Beasts

First Flight #49
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Submission Details
Impact: low
Likelihood: low
Invalid

Double Parameters Definition

Double Parameters Definition

Description

Openzeppelin Ownable is a well-auditted contract and takes the msg.sender as the owner even without the constructor.

contract BidBeastsNFTMarket is Ownable(msg.sender) {

Risk

Likelihood: Low

  • Reason 1 // WHEN an update is made on the Ownable contract by openzeppelin that doesn't take msg.sender, it might crash.

Recommended Mitigation

- remove this code
contract BidBeastsNFTMarket is Ownable(msg.sender) {
+ add this code
contract BidBeastsNFTMarket is Ownable() {
Updates

Lead Judging Commences

cryptoghost Lead Judge about 1 month ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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