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) Scope We audited the scroll-tech/scroll repository at the 2eb458c commit. contracts... We also performed a diff audit of the scroll-tech/scroll repository at the 2eb458c commit against 3bc8a3f commit. 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)