Submission Details
Impact:
Likelihood:
Use of too many mappings
Root + Impact
Description
The code uses too many mappaing object to hold values related to the pass.
should create a sing pass object/struct and the track them using one mapping.
// Pass data
mapping(uint256 => uint256) public passSupply; // Current minted
mapping(uint256 => uint256) public passMaxSupply; // Max allowed
mapping(uint256 => uint256) public passPrice;
Impact:
Not a very gas efficient design
Proof of Concept
Recommended Mitigation
- mapping(uint256 => uint256) public passSupply; // Current minted
- mapping(uint256 => uint256) public passMaxSupply; // Max allowed
- mapping(uint256 => uint256) public passPrice;
+ struct Pass{
uint256 supply;
uint256 maxSupply;
uint256 passPrice;
}
+ mapping(uint256 => Pass) passes
Rewards breakdown
nSLOC
234This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.