Skip to content

[ntuple] fix anchor decompression in mini file - #23940

Open
jblomer wants to merge 1 commit into
root-project:masterfrom
jblomer:ntuple-fix-minifile
Open

jblomer wants to merge 1 commit into
root-project:masterfrom
jblomer:ntuple-fix-minifile

Conversation

@jblomer

@jblomer jblomer commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Make the anchor decompression more robust by

  • Checking for invalid anchor size vs. length
  • Using the on-disk size for the input buffer (instead of the length)

Issue reported by Johnny Kao

@jblomer jblomer self-assigned this Nov 2, 2026
@Johnny-Kao

Copy link
Copy Markdown

@jblomer Thank you for raising this PR on my behalf! Really appreciate it.

@jblomer
jblomer force-pushed the ntuple-fix-minifile branch 2 times, most recently from 06dec35 to cd4c55d Compare November 2, 2026 09:13
Make the anchor decompression more robust by
  - Checking for invalid anchor size vs. length
  - Using the on-disk size for the input buffer (instead of the length)

Issue reported by Johnny Kao
@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown

Test Results

    21 files      21 suites   3d 10h 17m 25s ⏱️
 3 882 tests  3 534 ✅   0 💤 348 ❌
72 189 runs  70 659 ✅ 837 💤 693 ❌

For more details on these failures, see this check.

Results for commit cd4c55d.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants