Sablier

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

Two-step Owner transfer process should be implemented

Summary

Transferring a critical ownership role in a one step process is not advisable because a mistake of a wrong a address can brick the function forever and cannot be recovered.

Vulnerability Details

In Adminable.sol::transferAdmin() the current implementation jusr

Impact

Transferring ownership is a critical operation and transferring it to an inaccessible wallet or renouncing the owner-
ship e.g. by mistake, can effectively lost functionality

Tools Used

Manual review

Recommendations

implement a two step verification process

Updates

Lead Judging Commences

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