Beginner FriendlyDeFiFoundry
100 EXP
View results
Submission Details
Severity: high
Invalid

`Laundrette::retrieveAdmin` function is missing proper natspec about the functioning and it is not clear what the function is intended to do.

Summary

The Laundrette::retrieveAdmin , as the name suggests it should only retrieve who the Admin is but in its description it is changing the admin.

function retrieveAdmin() external {
kernel.executeAction(Actions.ChangeAdmin, kernel.executor());
}

Impact

As the name of the function and functioning suggest different things , wrong calling of the function maybe done which may cause the different action to be made instead of the desired action.

Tools Used

slither , aderyn , manual review

Recommendations

  1. A proper natspec should be provided about the function , what is it doing and what the different components are for.

  2. Name the function appropriately.

  3. The function should be made to only be called by the GodFather for now to ensure security and avoid any mishappening.

Updates

Lead Judging Commences

n0kto Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Design choice

Support

FAQs

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