Submission Details
Severity:
State Change Without Event
Summary
There are state variable changes in the setAdmin function but no event is emitted.
Vulnerability Details
The setAdmin function is responsible for setting a new admin but there is no event emitted.
Tools Used
Aderyn
Recommendations
Consider adding an event and emitting it after the admin has been changed.
+ event AdminChanged(address oldAdmin, address newAdmin);
setAdmin(address _newAdmin) external {
require(msg.sender == adminAddress, "Only admin can set new admin");
require(_newAdmin != address(0), "Admin cannot be zero address");
adminAddress = _newAdmin;
+ emit AdminChanged(oldAdmin, _newAdmin);
}
Updates
Appeal created
m3dython about 2 months ago
Submission Judgement Published Reason: Non-acceptable severity
Assigned finding tags:
Informational
Code suggestions or observations that do not pose a direct security risk.
m3dython about 2 months ago
Submission Judgement Published Reason: Non-acceptable severity
Assigned finding tags:
Informational
Code suggestions or observations that do not pose a direct security risk.
Rewards breakdown
nSLOC
349This 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.