This repository has been archived by the owner on May 26, 2023. It is now read-only.

seeu

low

Use safeTransferOwnership instead of the transferOwnership method

Summary

Use safeTransferOwnership instead of the transferOwnership method

Vulnerability Detail

In the contract, it was found the use of transferOwnership from @openzeppelin libraries.

Impact

It would be better to use safeTransferOwnership instead, which is a 2 structure.

Code Snippet

optimism/packages/contracts-bedrock/contracts/L1/SystemConfig.sol#L120
```
transferOwnership(_owner);
```

Tool used

Manual Review

Recommendation

It is advised to use safeTransferOwnership instead of the transferOwnership method. See Ownable2Step.sol.