DancingSeaHorse - Audit 2

CertiK26 Oct 2022Files named in the report

CertiK reviewed DancingSeaHorse (DancingSeaHorse - Audit 2) on 26 Oct 2022: 4 findings, 0 critical; 3 marked fixed, 1 acknowledged.

No open critical findings

Open the report Open DancingSeaHorse’s security record

Report hosted on skynet.certik.com.

Trustblock does not review code.

Findings4

Statuses are the security firm’s own, from its report.

Severity	Acknowledged	Fixed
Critical	0	0
High	1	0
Medium	0	0
Low	0	3

LowMissing Zero Address ValidationFixed

Addresses should be checked before assignment or external call to make sure they are not zero addresses. ```solidity=164 tokenAddress = _tokenAddress; ``` - `_tokenAddress` is not zero-checked before being used.

LowPotential Issue in Mint PriceFixed

`makeNft()` and `premint2Nft()` is using same variable `price` as the price of the NFT token.

LowState Variable Should Be Declared ConstantFixed

State variables that never change should be declared as `constant` to save gas. ```solidity=17 uint256 public premintPrice = 2 ether; ``` - `premintPrice` should be declared `constant`. --- ```solidity=18 address vaultAddress = 0x3C10D90796Ce828644951c0682012B3fabA52F8f; ``` - `vaultAddress` should be declared `constant`. --- ```solidity=19 uint256 public maxSupply = 8888; ``` - `maxSupply` should be declared `constant`. --- ```solidity=20 uint256 public maxMintSupplySale = 7000; ``` - `maxMintSupplySale` should be declared `constant`. --- ```solidity=31 uint256 public userLimit = 10; ``` - `userLimit` should be declared `constant`.