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

An address zero not checked

ixidor

Summary

  1. No check for address(0) in transferAdmin(address) - Adminable.sol

Vulnerability Details

1. No check for address(0) in transferAdmin(address) - Adminable.sol

The transferAdmin(address) (0x75829def) doesn't check that the next admin owner address is different than the null address.

Impact

1. No check for address(0) in transferAdmin(address) - Adminable.sol

Not checking the admin can result in a lock of the admin role because of the non-operable aspect of address(0).

Tools Used

  • slither

  • manual code analysis

  • foundry toolbox

Recommended Mitigation

1. No check for address(0) in transferAdmin(address) - Adminable.sol

The check can be performed in at several ways(require, revert statements) and in two different locations:

require(msg.sender != address(0), "next admin cannot be null");
Updates

Lead Judging Commences

inallhonesty Lead Judge over 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.