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
31	contracts/src/libraries/token/IScrollERC721.sol
32	contracts/src/L1/gateways/IL1ETHGateway.sol
33	contracts/src/libraries/verifier/IZkEvmVerifier.sol
34	contracts/src/L2/gateways/L2ERC721Gateway.sol
35	contracts/src/libraries/ScrollMessengerBase.sol
36	contracts/src/L1/rollup/L1MessageQueue.sol
37	contracts/src/L1/gateways/L1GatewayRouter.sol
38	contracts/src/gas-swap/GasSwap.sol
39	contracts/src/libraries/constants/ScrollConstants.sol
40	contracts/src/L1/gateways/IL1GatewayRouter.sol

#	File Name
31	contracts/src/libraries/token/IScrollERC721.sol
32	contracts/src/L1/gateways/IL1ETHGateway.sol
33	contracts/src/libraries/verifier/IZkEvmVerifier.sol
34	contracts/src/L2/gateways/L2ERC721Gateway.sol
35	contracts/src/libraries/ScrollMessengerBase.sol
36	contracts/src/L1/rollup/L1MessageQueue.sol
37	contracts/src/L1/gateways/L1GatewayRouter.sol
38	contracts/src/gas-swap/GasSwap.sol
39	contracts/src/libraries/constants/ScrollConstants.sol
40	contracts/src/L1/gateways/IL1GatewayRouter.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)