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
21	contracts/src/L1/gateways/L1CustomERC20Gateway.sol
22	contracts/src/L1/rollup/MultipleVersionRollupVerifier.sol
23	contracts/src/libraries/callbacks/IMessageDropCallback.sol
24	contracts/src/L1/rollup/ScrollChain.sol
25	contracts/src/L1/rollup/IL1MessageQueue.sol
26	contracts/src/L1/IL1ScrollMessenger.sol
27	contracts/src/libraries/gateway/ScrollGatewayBase.sol
28	contracts/src/L1/gateways/L1WETHGateway.sol
29	contracts/src/L1/gateways/L1ETHGateway.sol
30	contracts/src/libraries/token/IScrollERC1155.sol

#	File Name
21	contracts/src/L1/gateways/L1CustomERC20Gateway.sol
22	contracts/src/L1/rollup/MultipleVersionRollupVerifier.sol
23	contracts/src/libraries/callbacks/IMessageDropCallback.sol
24	contracts/src/L1/rollup/ScrollChain.sol
25	contracts/src/L1/rollup/IL1MessageQueue.sol
26	contracts/src/L1/IL1ScrollMessenger.sol
27	contracts/src/libraries/gateway/ScrollGatewayBase.sol
28	contracts/src/L1/gateways/L1WETHGateway.sol
29	contracts/src/L1/gateways/L1ETHGateway.sol
30	contracts/src/libraries/token/IScrollERC1155.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)