Skip to content

[unstable2507] Backport #9127#9237

Merged
acatangiu merged 2 commits intounstable2507from
backport-9127-to-unstable2507
Jul 17, 2025
Merged

[unstable2507] Backport #9127#9237
acatangiu merged 2 commits intounstable2507from
backport-9127-to-unstable2507

Conversation

@paritytech-release-backport-bot
Copy link
Copy Markdown

Backport #9127 into unstable2507 from alexggh.

See the documentation on how to use this bot.

…lidation context (#9127)

#8606
paritytech/trie#221 replaced the usage of
BTreeMap with HashMaps in validation context. The keys are already
derived with a cryptographic hash function from user data, so users
should not be able to manipulate it.

To be on safe side this PR also modifies the TrieCache, TrieRecorder and
MemoryDB to use a hasher that on top of the default generated randomness
also adds randomness generated from the hash of the relaychain and that
of the parachain blocks, which is not something users can control or
guess ahead of time.

---------

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
Co-authored-by: Bastian Köcher <git@kchr.de>
Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
(cherry picked from commit 7058819)
@paritytech-release-backport-bot paritytech-release-backport-bot bot requested a review from a team as a code owner July 17, 2025 07:20
@github-actions github-actions bot added the A3-backport Pull request is already reviewed well in another branch. label Jul 17, 2025
@github-actions github-actions bot requested a review from alexggh July 17, 2025 07:20
@alexggh alexggh removed the request for review from a team July 17, 2025 07:22
@acatangiu acatangiu enabled auto-merge (squash) July 17, 2025 12:12
@acatangiu acatangiu merged commit ffaa6ed into unstable2507 Jul 17, 2025
269 of 272 checks passed
@acatangiu acatangiu deleted the backport-9127-to-unstable2507 branch July 17, 2025 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A3-backport Pull request is already reviewed well in another branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants