Skip to content

[io] fix memleak in TFile::Recover UPDATE mode#21999

Merged
dpiparo merged 1 commit intoroot-project:masterfrom
ferdymercury:patch-17
Apr 22, 2026
Merged

[io] fix memleak in TFile::Recover UPDATE mode#21999
dpiparo merged 1 commit intoroot-project:masterfrom
ferdymercury:patch-17

Conversation

@ferdymercury
Copy link
Copy Markdown
Collaborator

@ferdymercury ferdymercury commented Apr 21, 2026

This Pull request:

Changes or fixes:

Fixes #21997

thanks Vincenzo for opening the issue :)

Checklist:

  • tested changes locally
  • updated the docs (if necessary)

@ferdymercury ferdymercury requested a review from pcanal as a code owner April 21, 2026 13:35
@ferdymercury ferdymercury requested a review from vepadulano April 21, 2026 13:35
@vepadulano vepadulano self-assigned this Apr 21, 2026
Copy link
Copy Markdown
Member

@vepadulano vepadulano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but let's also wait for the CI to pass and for @pcanal's review

Copy link
Copy Markdown
Member

@pcanal pcanal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks.

@dpiparo dpiparo added this to the 6.40.00-rc1 milestone Apr 21, 2026
@github-actions
Copy link
Copy Markdown

Test Results

    22 files      22 suites   3d 12h 16m 54s ⏱️
 3 838 tests  3 836 ✅  1 💤  1 ❌
75 727 runs  75 698 ✅ 18 💤 11 ❌

For more details on these failures, see this check.

Results for commit eaa3360.

@dpiparo dpiparo merged commit a690146 into root-project:master Apr 22, 2026
18 of 30 checks passed
@dpiparo
Copy link
Copy Markdown
Member

dpiparo commented Apr 22, 2026

/backport to 6.38

@root-project-bot
Copy link
Copy Markdown

This PR has been backported to branch 6.38: #22009

@ferdymercury ferdymercury deleted the patch-17 branch April 22, 2026 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Memory leak in TFile::Open with UPDATE mode

5 participants