GivingThanks

First Flight #28
Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: high
Valid

[H-3] lack of access control in updateRegistry::GivingThanks.sol

Summary

updateRegistry lacks access control therefore any user could update the registry in any malicious ways

@>function updateRegistry(address _registry) public {
registry = CharityRegistry(_registry);
}

Impact

malicious user can update the registry to perform malicious acts

Tools Used

Manual review

Recommendations

updateRegistry should be given access control, that can called by trusted entities in the protocol

Updates

Lead Judging Commences

n0kto Lead Judge 8 months ago
Submission Judgement Published
Validated
Assigned finding tags:

finding-anyone-can-change-registry

Likelyhood: High, anyone can change it at anytime Impact: High, can bypass the verification process

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.