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

Bug: invalid G1 and G2 cached in PrivateKey #11

Merged
merged 2 commits into from Nov 26, 2021

Conversation

random-zebra
Copy link

Pretty serious bug in the copy-assignment operator, discovered after a long investigation with @furszy over an intermittent failure in a unit test for deterministic masternodes in Core.

For reference, pull request submitted upstream: Chia-Network#289

Copy link

@furszy furszy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

obvious ACK 183be27 🤘 . It was already merged in upstream as well so.. merging..

@furszy furszy merged commit eaf7d2d into PIVX-Project:main Nov 26, 2021
furszy added a commit to furszy/bitcoin-core that referenced this pull request Nov 26, 2021
eaf7d2d1a4 Merge PIVX-Project/bls-signatures#11: Bug: invalid G1 and G2 cached in PrivateKey
183be2750c BugFix: Invalidate caches in PrivateKey copy assignment operator
427246653c QA: test cache invalidation for PrivateKey copy assignment operator

git-subtree-dir: src/chiabls
git-subtree-split: eaf7d2d1a49115c8d098f96f80b5db9aa487b1d5
furszy added a commit to PIVX-Project/PIVX that referenced this pull request Dec 1, 2021
bc84ff0 Doc: add bls-sigs repo to developer-notes subtrees section (furszy)
9daeb29 Squashed 'src/chiabls/' changes from bc64f128f0..eaf7d2d1a4 (furszy)

Pull request description:

  Updating the bls-signatures subtree, reason is PIVX-Project/bls-signatures#11 (which gladly was detected before v6.0).
  Coming from #2631 investigation.

  Pulling up to PIVX-Project/bls-signatures@eaf7d2d.

ACKs for top commit:
  random-zebra:
    utACK bc84ff0

Tree-SHA512: 91744a317eb8834bdac792a2cfc0fca306bb63798ec037743057c87238087c68b2bceca5811d6fdb9865a8a3ba5f5dd7268bebd398bc2d80223cd3e1baa625bd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants