Doc: add a comment referencing past vulnerability next to where it was fixed #30538

pull darosior wants to merge 6 commits into bitcoin:master from darosior:2407_doc_reference_past_vulns changing 5 files +20 −0
  1. darosior commented at 9:11 am on July 28, 2024: member
    It is useful when reading code to have context about why it is written or behaves the way it does. Some instances in this PR may seem obvious but i think nonetheless offer important context to anyone willing to change (or review a change to) this code.
  2. doc: validation: add a reference to historical header spam vulnerability d6b46d8d5e
  3. doc: net_proc: reference past defect regarding invalid GETDATA types 7b44f0741f
  4. doc: net_proc: reference past DoS vulnerability in orphan processing 382f67d39a
  5. doc: txrequest: point to past censorship vulnerability in tx re-request handling f57f621638
  6. doc: net: mention past vulnerability as rationale to limit incoming message size 70992f59de
  7. doc: banman: reference past vuln due to unbounded banlist 7fea8eeeb9
  8. DrahtBot commented at 9:11 am on July 28, 2024: contributor

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

    Code Coverage

    For detailed information about the code coverage, see the test coverage report.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK tdb3, mjdietzx

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

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #30110 (refactor: TxDownloadManager + fuzzing by glozow)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  9. DrahtBot added the label Docs on Jul 28, 2024
  10. tdb3 approved
  11. tdb3 commented at 1:30 pm on July 28, 2024: contributor
    ACK 7fea8eeeb9984ff6f3ed661f3970b1aaa68548de Great idea Tested links (worked)
  12. eval-exec approved
  13. mjdietzx commented at 4:52 pm on July 29, 2024: contributor
    ACK 7fea8eeeb9984ff6f3ed661f3970b1aaa68548de
  14. glozow requested review from dergoegge on Jul 31, 2024
  15. achow101 commented at 11:15 pm on September 17, 2024: member

    ~0

    I’m not sure that having a comment that references code that no longer exists is all that useful.

  16. DrahtBot added the label CI failed on Oct 13, 2024
  17. DrahtBot removed the label CI failed on Oct 19, 2024
  18. DrahtBot added the label Needs rebase on Oct 29, 2024
  19. DrahtBot commented at 8:05 pm on October 29, 2024: contributor

    🐙 This pull request conflicts with the target branch and needs rebase.


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-11-21 12:12 UTC

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