contrib: [tracing] fix pointer argument handling in mempool_monitor.py #33086

pull deadmanoz wants to merge 1 commits into bitcoin:master from deadmanoz:fix-mempool-monitor-hash-reason-pointers changing 1 files +7 −7
  1. deadmanoz commented at 8:39 am on July 29, 2025: contributor

    The BPF code was incorrectly passing pointer variables by value to bpf_usdt_readarg(), causing the function to fail silently and resulting in transaction hashes showing as zeros and reason strings displaying empty strings.

    This fix adds the missing reference operator (&) when passing pointer variables to bpf_usdt_readarg(), allowing the function to properly write the pointer values and enabling correct display of transaction hashes and removal/rejection reasons.

    Fixes the regression introduced in ec47ba349d where bpf_usdt_readarg_p was replaced with bpf_usdt_readarg but the calling convention wasn’t properly updated for pointer arguments.

    Before: “0000000000000000000000000000000000000000000000000000000000000000” tx hashes, and missing reasons (empty strings) for removal.

    After: tx hashes show, reasons for removal showing.

  2. tracing: fix pointer argument handling in mempool_monitor.py
    The BPF code was incorrectly passing pointer variables by value to
    bpf_usdt_readarg(), causing the function to fail silently and resulting
    in transaction hashes and reason strings displaying as zeros or garbage.
    
    This fix adds the missing reference operator (&) when passing pointer
    variables to bpf_usdt_readarg(), allowing the function to properly
    write the pointer values and enabling correct display of transaction
    hashes and removal/rejection reasons.
    
    Fixes the regression introduced in ec47ba349d where bpf_usdt_readarg_p
    was replaced with bpf_usdt_readarg but the calling convention wasn't
    properly updated for pointer arguments.
    0ce041ea88
  3. DrahtBot added the label Scripts and tools on Jul 29, 2025
  4. DrahtBot commented at 8:39 am on July 29, 2025: contributor

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

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33086.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK 0xB10C

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

  5. fanquake requested review from 0xB10C on Jul 29, 2025
  6. 0xB10C commented at 9:26 am on July 29, 2025: contributor

    tested ACK 0ce041ea88dbea2bdfaf21fd0e60a86012498f0a

    Thanks! I can reproduce the problem and confirm that this fixes it.

  7. fanquake added the label Needs backport (29.x) on Jul 29, 2025
  8. fanquake merged this on Jul 29, 2025
  9. fanquake closed this on Jul 29, 2025

  10. fanquake referenced this in commit 9437415024 on Jul 29, 2025
  11. fanquake commented at 9:56 am on July 29, 2025: member
    Backported to 29.x in #33074.
  12. deadmanoz deleted the branch on Jul 29, 2025
  13. fanquake removed the label Needs backport (29.x) on Jul 29, 2025

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: 2025-08-02 09:13 UTC

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