test: add test for corrupt wallet bdb logs #20974

pull ryanofsky wants to merge 3 commits into bitcoin:master from ryanofsky:pr/badbdb changing 4 files +37 −16
  1. ryanofsky commented at 4:41 am on January 21, 2021: contributor
    This PR adds a wallet test testing handling of corrupt BDB logs. It restores commit c6a2c265f5d0b424fc8226c2f3426e82856779d0 from #18907 which was dropped because it triggered a memory leak, which is reported as #19034
  2. ryanofsky commented at 4:44 am on January 21, 2021: contributor
    Opened as draft PR to see if tests will pass now. If they do then #19034 can be closed and this can be reviewed and merged. Otherwise some more debugging will be required.
  3. fanquake added the label Tests on Jan 21, 2021
  4. DrahtBot commented at 2:32 am on January 26, 2021: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    Approach ACK jonasschnelli

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    Conflicts

    No conflicts as of last run.

  5. jonasschnelli commented at 8:52 am on January 26, 2021: contributor
    Approach ACK
  6. laanwj requested review from achow101 on Jan 27, 2021
  7. DrahtBot added the label Needs rebase on Dec 13, 2021
  8. uvhw referenced this in commit 47d44ccc3e on Feb 14, 2022
  9. test: add test for corrupt wallet bdb logs 186f00b333
  10. attempt fix memory leak 75534c7ae9
  11. basically revert but remove reset mehtod 250860e2ae
  12. ryanofsky force-pushed on Jan 20, 2023
  13. DrahtBot removed the label Needs rebase on Jan 20, 2023
  14. achow101 commented at 2:47 pm on May 3, 2023: member
    I’m not sure that the issue this is trying to fix is really an issue worth spending time on since we have BDB slated for removal.
  15. achow101 commented at 4:48 pm on September 23, 2023: member
    Closing as this has been open as a draft for more than 2 years with basically no activity. The tests do not pass, nor does it appear that the issue has been fixed or will be fixed. The planned removal of BDB will also obsolete this test.
  16. achow101 closed this on Sep 23, 2023

  17. bitcoin locked this on Sep 22, 2024

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-12-18 18:12 UTC

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