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

Add zero address checks for L*BitcoinDepositor initializers #805

Merged
merged 2 commits into from
Apr 3, 2024

Conversation

lukasz-zimnoch
Copy link
Member

@lukasz-zimnoch lukasz-zimnoch commented Apr 3, 2024

Depends on: #804

Here we add the missing zero-address checks to prevent incorrectly set values.

Here we add the missing zero-address checks to prevent incorrectly set values.
@lukasz-zimnoch lukasz-zimnoch marked this pull request as draft April 3, 2024 12:16
@lukasz-zimnoch lukasz-zimnoch self-assigned this Apr 3, 2024
@lukasz-zimnoch lukasz-zimnoch added the ⛓️ solidity Solidity contracts label Apr 3, 2024
Base automatically changed from authorized-refunds to main April 3, 2024 12:24
@lukasz-zimnoch lukasz-zimnoch marked this pull request as ready for review April 3, 2024 12:30
@tomaszslabon tomaszslabon enabled auto-merge April 3, 2024 12:33
@tomaszslabon tomaszslabon merged commit 26f87a6 into main Apr 3, 2024
37 checks passed
@tomaszslabon tomaszslabon deleted the address-checks branch April 3, 2024 12:36
tomaszslabon added a commit that referenced this pull request Apr 3, 2024
Depends on: #805

Here we fix the Solidity version to be `0.8.17` which is used across all
tBTC contracts.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⛓️ solidity Solidity contracts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants