Sablier

Sablier
DeFiFoundry
53,440 USDC
View results
Submission Details
Severity: low
Invalid

Old Admin check missing

Summary

No previous admin check present

Impact

Any old admin can set themselves to be the newAdmin again

Tools Used

Manual Review

Recommendations

function transferAdmin(address newAdmin) public virtual override onlyAdmin {
// Effect: update the admin.
- admin = newAdmin;
+ require(admin != newAdmin,"");
// Log the transfer of the admin.
emit IAdminable.TransferAdmin({ oldAdmin: msg.sender, newAdmin: newAdmin });
}
Updates

Lead Judging Commences

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Info/Gas/Invalid as per Docs

https://docs.codehawks.com/hawks-auditors/how-to-determine-a-finding-validity

Support

FAQs

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