Skip to content

(release/25.0) rootless: Fix two region leaks - #3606

Open
metux wants to merge 1 commit into
release/25.0from
pr/release/25.0-rootless-fix-two-region-leaks-_2026-08-20_16-24-51
Open

(release/25.0) rootless: Fix two region leaks#3606
metux wants to merge 1 commit into
release/25.0from
pr/release/25.0-rootless-fix-two-region-leaks-_2026-08-20_16-24-51

Conversation

@metux

@metux metux commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

RootlessEnsureFrame returns without releasing a shaped window's region when the implementation refuses to create the
frame, and RootlessMiValidateTree never releases the exposed region it hands to RootlessComputeClips -- upstream
miValidateTree does.

Both are longstanding.

Signed-off-by: Jeremy Huddleston Sequoia jeremyhu@apple.com

RootlessEnsureFrame returns without releasing a shaped window's region when the implementation refuses to create the
frame, and RootlessMiValidateTree never releases the exposed region it hands to RootlessComputeClips -- upstream
miValidateTree does.

Both are longstanding.

Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>
@metux metux self-assigned this Aug 20, 2026
@metux
metux requested a review from a team August 20, 2026 14:26
metux pushed a commit that referenced this pull request Aug 20, 2026
RootlessEnsureFrame returns without releasing a shaped window's region when the implementation refuses to create the
frame, and RootlessMiValidateTree never releases the exposed region it hands to RootlessComputeClips -- upstream
miValidateTree does.

Both are longstanding.

Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>
PR: #3606
@metux

metux commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

🤖 Automated review — generated by Starfleet ship Voyager on behalf of @metux. Not a human review.

Identical cherry-pick of PR #3593 (already reviewed on release/25.1). Same two rootless region leak fixes — RootlessMiValidateTree missing RegionUninit(&exposed), RootlessEnsureFrame missing RegionUninit(&shape) on the early-return path. ABI-safe. Correct for release/25.0.

@metux metux added the bot-review-passed Automated bot review found no blocking issues label Aug 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot-review-passed Automated bot review found no blocking issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants