Crash debug builds when mempool ConsensusScriptChecks fails #23590

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2111-crashMempool changing 1 files +2 −2
  1. MarcoFalke commented at 10:47 am on November 25, 2021: member

    Currently a bug in the function might sneak around our testing infrastructure.

    Fix that by turning bugs into crashes during tests.

  2. Crash debug builds when mempool ConsensusScriptChecks fails faad05c6d2
  3. MarcoFalke added the label Refactoring on Nov 25, 2021
  4. MarcoFalke force-pushed on Nov 25, 2021
  5. glozow commented at 8:28 am on November 29, 2021: member
    utACK faad05c6d2ad1840c26f6fdea7d358bf9ba6efea, there’s something seriously wrong with the code if this returns false, good to throw in debug mode
  6. luke-jr approved
  7. luke-jr commented at 10:28 pm on November 30, 2021: member
    utACK
  8. MarcoFalke merged this on Dec 1, 2021
  9. MarcoFalke closed this on Dec 1, 2021

  10. MarcoFalke deleted the branch on Dec 1, 2021
  11. sidhujag referenced this in commit 8fb18452a7 on Dec 1, 2021
  12. RandyMcMillan referenced this in commit c4e2cc8337 on Dec 23, 2021
  13. Fabcien referenced this in commit d2f5796b2d on Nov 7, 2022
  14. DrahtBot locked this on Dec 1, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-07-03 10:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me