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
Backport of bitcoin/bitcoin#8020 #4979
Conversation
This is ~1.7x slower than the Lookup3-of-Xor-with-salt construct we were using before, but it is a primitive designed for exactly this. Zcash: Propagate CCoinsKeyHasher -> SaltedTxidHasher changes to where we've used CCoinsKeyHasher outside of its original scope.
Zcash: conflicts resolved against 957e5d2
How does this conflict? 5490 from upstream was merged before we forked. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK. I checked the SipHash-2-4 implementation against the reference implementation.
This does not affect consensus (it alters the hasher used inside maps that are used by consensus, but that only affects the DoS-resistance of those maps, not their consensus behaviour), or risk user funds, and thus only requires one review. @zkbot r+ |
📌 Commit cfc4ca1 has been approved by |
Cherry-picked from bitcoin/bitcoin#8020