GivingThanks

First Flight #28
Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: low
Invalid

Admin can unintentionally set newAdmin to address(0) making the CharityRegistry null.

Summary

in the changeAdmin(), admin may pass an address(0) making the contract obsolete.

Vulnerability Details

Impact

Tools Used

Recommendations

Add an check statement before updating newAdmin as :

function changeAdmin(address newAdmin) public {
require(msg.sender == admin, "Only admin can change admin");
require(newAdmin != address(0), "Invalid admin address");
admin = newAdmin;
}
Updates

Lead Judging Commences

n0kto Lead Judge 8 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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