code-423n4 / 2022-01-livepeer-findings

0 stars 0 forks source link

Optimisation #182

Closed code423n4 closed 2 years ago

code423n4 commented 2 years ago

Handle

0v3rf10w

Vulnerability details

Impact

Dead code

Proof of Concept

L1ArbitrumMessenger.sendTxToL2(address,address,uint256,uint256,uint256,bytes) (contracts/L1/gateway/L1ArbitrumMessenger.sol#34-53)
L1ArbitrumMessenger.sendTxToL2(address,address,uint256,uint256,uint256,uint256,uint256,bytes) (contracts/L1/gateway/L1ArbitrumMessenger.sol#55-77)

call graph shows these are not used and thus need to be removed

Tools Used

Vscode solidity dev extension

Recommended Mitigation Steps

Remove dead code

yondonfu commented 2 years ago

Both functions are used in the contracts in the L1/gateway director of arbitrum-lpt-bridge.