Biconomy: Nexus
Biconomy
HardhatFoundry
30,000 USDC
View results
Previous Next
Submission Details
Severity: low
Invalid

Immutable address `_SELF` is not used in `Nexus.sol` and can be removed

sammy

Summary

_SELFis an immutable address that is set in the Nexus.solcontract's constructor. It isn't used anywhere in the contract and can be removed from the code to save storage and gas costs.

address private immutable _SELF;
​
/// @dev `keccak256("PersonalSign(bytes prefixed)")`.
bytes32 internal constant _PERSONAL_SIGN_TYPEHASH = 0x983e65e5148e570cd828ead231ee759a8d7958721a768f93bc4483ba005c32de;
​
/// @notice Initializes the smart account with the specified entry point.
constructor(address anEntryPoint) {
_SELF = address(this);
require(address(anEntryPoint) != address(0), EntryPointCanNotBeZero());
_ENTRYPOINT = anEntryPoint;
_initModuleManager();
}

Vulnerability Details

Impact

Gas/ Storage optimization

Tools Used

Manual Review

Recommendations

Remove _SELFfrom the code

Updates

Lead Judging Commences

0xnevi Lead Judge
12 months ago
0xnevi Lead Judge 11 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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