Submission Details
Severity:
Anybody can call changeOwner function and be the owner.
Summary
changeOwner function has no guard, as a result any body can call changeOwner function and pass his address to make himself the owner.
Vulnerability Details
function changeOwner(address _newOwner) public {
owner = _newOwner;
}
Impact
A malicious can make himself the owner by calling changeOwner function and withdraw all funds from the contract.
Tools Used
Manual Review
Recommendations
add guard to the changeOwner function.
function changeOwner(address _newOwner) public {
require(msg.sender == owner, "Only current owner can make new owner");
owner = _newOwner;
}
Updates
Appeal created
inallhonesty 11 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.