Submission Details
Severity:
changeOwner() can be called by anyone
Summary
Anyone can call change owner, and then withdraw all funds
Vulnerability Details
Malicious user call `changeOwner(maliciousAdress)`
Malicious user call `withdrawFunds()`
Impact
Anyone can steal all Ether on the contract
Recommendations
Add a check so that only the owner can change the owner of the contract :
function changeOwner(address _newOwner) public {
+ require(msg.sender == owner, "Only owner can change owner");
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.