Submission Details
Severity:
No check for already registered mapping
Summary
No check for already registered mapping
Vulnerability Details
CharityRegistry::registerCharitydoes not check if the charity has already been registered.
Impact
user can register charity multiple times
Tools Used
Manual Review
Recommendations
Implement the function like this :
function registerCharity(address charity) public {
+ require(!registeredCharities[charity], "Charity is already registered");
registeredCharities[charity] = true;
}
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.