Submission Details
Severity:
No access control for function MysteryBox::changeOwner allows anyone to withdraw all funds from the contract
Summary
Anyone can change owner of the contract and then withdraw all funds from the contract.
Vulnerability Details
Function MysteryBox::changeOwner does not have any access control in place. Therefore anyone can call the function and change the contract owner. After that the attacker can call function MysteryBox::withdrawFunds and withdraw all funds from the contract.
Impact
Anyone can withdraw all funds from the contract.
Tools Used
Manual review
Recommendations
Add permissions check to the function MysteryBox::changeOwner as show below:
function changeOwner(address _newOwner) public {
+ require(msg.sender == owner, "Only owner can change owner");
owner = _newOwner;
}
Updates
Appeal created
inallhonesty about 1 year 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.