Submission Details
Severity:
Anyone can be owner to steal all funds.
Summary
The changeOwner() has no restrictions, so anyone can change owner to steall all funds.
Vulnerability Details
The changeOwner() function does not include an ownership check or any form of access control. This means that any user who interacts with the contract can call this function and change the contract's owner to any address, including their own.
Impact
Malicious users can take over the contract by setting themselves as the owner and steal all funds.
Tools Used
Manual Review
Recommendations
function changeOwner(address _newOwner) public onlyOwner {
+ require(msg.sender == owner, "Only owner can execute this");
owner = _newOwner;
}
Updates
Appeal created
inallhonesty 9 months ago
Submission Judgement Published Assigned finding tags:
Anyone can change owner
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.