Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

contracts: Apply depth limit when decoding #11991

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

athei
Copy link
Member

@athei athei commented Aug 8, 2022

This applies the current depth limit of 256 to the decoding in pallet-contracts of contract generated data. We do not rely on MAX_EXTRINSIC_DEPTH because we cannot risk that this value will be changed in the future and hence break existing contracts.

@athei athei added A0-please_review Pull request needs code review. B7-runtimenoteworthy C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. labels Aug 8, 2022
@athei
Copy link
Member Author

athei commented Aug 8, 2022

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit 74a6370 into master Aug 8, 2022
@paritytech-processbot paritytech-processbot bot deleted the at/depth-limit branch August 8, 2022 14:32
DaviRain-Su pushed a commit to octopus-network/substrate that referenced this pull request Aug 23, 2022
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants