Submission Details
Severity:
Missing access control
Summary
Missing access control
Vulnerability Details
Missing access control for changing owner. Anyone can become owner and steal the rewards.
https://github.com/Cyfrin/2024-09-mystery-box/blob/281a3e35761a171ba134e574473565a1afb56b68/src/MysteryBox.sol#L111C1-L113C6
Impact
Owner can be changed without validation.
Tools Used
MR
Recommendations
Only allow current owner to change address
function changeOwner(address _newOwner) public {
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.