Submission Details
Severity:
lacks a zero-check for CharityRegistry::changeAdmin
Summary
The address 0 has not been checked in the changeAdmin() function. If the address 0 is sent, it will be considered as the admin and the admin can no longer be changed.
Vulnerability Details
There will be a lack of access to the parts that the admin can do, and the program will be without an admin
Impact
Tools Used
personal Information
Recommendations
function changeAdmin(address newAdmin) public {
+ require(newAdmin!=address(0));
require(msg.sender == admin, "Only admin can change admin");
admin = newAdmin;
}
Rewards breakdown
nSLOC
67This 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.