First Flight #21: KittyFi

First Flight #21
Beginner FriendlyDeFiFoundry
100 EXP
View results
Submission Details
Severity: high
Invalid

Lack of Validation for Token Address in `KittyVault::constructor` function

Description: The constructor does not validate the _token, _kittyPool, _priceFeed, _euroPriceFeed, _meowntainer, and _aavePool addresses, which could lead to setting invalid or malicious addresses.

Impact: Invalid or malicious addresses could lead to loss of funds or other unexpected behavior.

Recommended Mitigation: Add validation checks to ensure that addresses are non-zero and valid.

constructor(
address _token,
address _kittyPool,
address _priceFeed,
address _euroPriceFeed,
address _meowntainer,
address _aavePool
) {
require(_token != address(0), "Invalid token address");
require(_kittyPool != address(0), "Invalid kitty pool address");
require(_priceFeed != address(0), "Invalid price feed address");
require(_euroPriceFeed != address(0), "Invalid euro price feed address");
require(_meowntainer != address(0), "Invalid meowntainer address");
require(_aavePool != address(0), "Invalid aave pool address");
token = _token;
kittyPool = _kittyPool;
priceFeed = _priceFeed;
euroPriceFeed = _euroPriceFeed;
meowntainer = _meowntainer;
aavePool = _aavePool;
}
Updates

Lead Judging Commences

shikhar229169 Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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