Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SC-430] Revert on non-existent contract instead of silent success #17

Merged
merged 4 commits into from
Jun 11, 2024

Conversation

hexonaut
Copy link
Contributor

@hexonaut hexonaut commented Jun 8, 2024

Switched to using the OZ Address helper library which does an additional check for empty addresses and reverts if so. This will help with things like cross-chain payload e2e testing to have them fail when the address is not deployed. Also added some more testing coverage for these types of failure cases with delegatecall and call.

… add some more testing coverage for revert cases
barrutko
barrutko previously approved these changes Jun 10, 2024
test/AuthBridgeExecutor.t.sol Outdated Show resolved Hide resolved
test/AuthBridgeExecutor.t.sol Outdated Show resolved Hide resolved
Copy link

Coverage after merging SC-430-error-on-empty-contract into master will be

100.00%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src/executors
   AuthBridgeExecutor.sol100%100%100%100%
   BridgeExecutorBase.sol100%100%100%100%
src/receivers
   BridgeExecutorReceiverArbitrum.sol100%100%100%100%
   BridgeExecutorReceiverGnosis.sol100%100%100%100%
   BridgeExecutorReceiverOptimism.sol100%100%100%100%

@hexonaut hexonaut merged commit 95c3bb8 into master Jun 11, 2024
3 checks passed
@hexonaut hexonaut deleted the SC-430-error-on-empty-contract branch June 11, 2024 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants