code-423n4 / 2021-05-nftx-findings

1 stars 0 forks source link

struct Config #34

Closed code423n4 closed 3 years ago

code423n4 commented 3 years ago

Handle

paulius.eth

Vulnerability details

Impact

There are 5 structs named "Config" which are not used anywhere. For example, in contract NFTXMintRequestEligibility: struct Config { address owner; address vaultAddress; bool reverseEligOnRedeem; uint256[] tokenIds; }

Recommended Mitigation Steps

Remove unused structs or use it where intended.