Scroll GasSwap, Multiple Verifier, Wrapped Ether a

Off-Chain (Private)

Audited on 2023/10/16

No active critical issues

Summary

2 partially resolved) Low Severity Issues 10 (8 resolved) Notes & Additional Information 15 (5 resolved, 2 partially resolved) Client Reported Issues 0 (0 resolved) We audited the scroll-tech/scroll repository at the 2eb458c commit. We also performed a diff audit of the scroll-tech/scroll repository... to be used for different batches. A GasSwap contract was also added to help users swap tokens for L2 ETH. Aside from these major changes, this audit focused on diff audits across almost all contracts across this protocol. In this audit, we aimed to verify the correctness and security of the contracts...

Project

Scroll

Multi-Chain

Issues (17)

	Low	Medium	High	Critical	Total
Not fixed	-	2	-	-	2
Fixed	10	3	2	-	15
Total	10	5	2	0	17

Contracts (45)

#	File Name
41	contracts/src/L2/predeploys/L1GasPriceOracle.sol
42	contracts/src/L2/gateways/usdc/L2USDCGateway.sol
43	contracts/src/L2/predeploys/WrappedEther.sol
44	contracts/src/L1/gateways/IL1ERC721Gateway.sol
45	contracts/src/L1/gateways/IL1ERC1155Gateway.sol

#	File Name
41	contracts/src/L2/predeploys/L1GasPriceOracle.sol
42	contracts/src/L2/gateways/usdc/L2USDCGateway.sol
43	contracts/src/L2/predeploys/WrappedEther.sol
44	contracts/src/L1/gateways/IL1ERC721Gateway.sol
45	contracts/src/L1/gateways/IL1ERC1155Gateway.sol

Scroll GasSwap, Multiple Verifier, Wrapped Ether a

Summary

Issues (17)

Potentially Stuck ETH from Incorrect Data Parameternot_fixed/medium

Potentially Stuck ETH in L1ScrollMessengernot_fixed/medium

Contracts (45)