code-423n4 / 2021-12-amun-findings

0 stars 0 forks source link

`PolygonERC20Wrapper.sol` Use immutable variable can save gas #204

Closed code423n4 closed 2 years ago

code423n4 commented 2 years ago

Handle

WatchPug

Vulnerability details

https://github.com/code-423n4/2021-12-amun/blob/98f6e2ff91f5fcebc0489f5871183566feaec307/contracts/bridge/contracts/amunBasketBridge/PolygonERC20Wrapper.sol#L13-L14

Considering that underlying, childChainManager will never change, changing them to immutable variables instead of storages variable can save gas.

0xleastwood commented 2 years ago

Duplicate of #18