Submission Details
Severity:
`Trustee:setAssetToPay` lacks proper checks
Summary
Due to protocol specifications, the asset cannot be Ether; otherwise, the buyOutEstateNFT function will fail due to ERC20 usage
Vulnerability Details
In the setAssetToPay function, the passed asset is assigned directly without proper checks
function setAssetToPay(address _asset) external onlyTrustee {
assetToPay = _asset;
}
Impact
Due to the configuration of buyOutEstateNFT, which operates only with ERC20 tokens, this can lead to a loss of funds
Recommendations
Implement a check before assigning the asset and revert if it is Ether
function setAssetToPay(address _asset) external onlyTrustee {
+ require(_asset != address(0), "Asset cannot be Ether");
assetToPay = _asset;
}
Rewards breakdown
nSLOC
211This 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.