Submission Details
Severity:
Missing `address(0)` checks
Summary
Missing validation for address(0) when assigning values to address state variables.
Vulnerability Details
In the changeOwner() function, there is no check to prevent setting the owner to addres(0). As a result, the owner can be updated to a zero address.
Impact
This could lead to a loss of contract ownership, as the user might unintentionally or maliciously transfer ownership to address(0), making the contract inaccessible.
Tools Used
Manual review, Slither
Recommendations
Consider adding address(0) checks for _newOwner:
function changeOwner(address _newOwner) public {
+ require(_newOwner != address(0));
owner = _newOwner;
}
Rewards breakdown
nSLOC
86This 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.