Skip to content

[qt6web] fix crash during application teardown - #23794

Merged
linev merged 3 commits into
root-project:masterfrom
linev:qt6web_fix
Oct 23, 2026
Merged

linev merged 3 commits into
root-project:masterfrom
linev:qt6web_fix

Conversation

@linev

@linev linev commented Oct 22, 2026

Copy link
Copy Markdown
Member

During application shutdown ROOT libraries
can be partially unloaded and therefore
deletion of timer object registered in the global lists may cause application crash.
Same is true for qt6 Url scheme handler.

Therefore when gSystem is nullptr - skip objects cleanup

Reduce default timer period for Qt6 web canvas to 10 ms.

Update documentation.

linev added 3 commits October 22, 2026 12:58
During application shutdown ROOT libraries
can be partially unloaded and therefore
deletion of timer object registered in the global lists may cause application crash.
Same is true for qt6 Url scheme handler.

Therefore when gSystem is nullptr - skip objects cleanup
From 1 to 10 ms, change name to WebGui.Qt6Timer, was not documented before
@linev
linev requested a review from bellenot October 22, 2026 11:00
@linev linev self-assigned this Oct 22, 2026

@bellenot bellenot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@github-actions

github-actions Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Test Results

    24 files      24 suites   4d 4h 32m 21s ⏱️
 3 895 tests  3 894 ✅ 0 💤 1 ❌
83 365 runs  83 364 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit fbf5bf2.

♻️ This comment has been updated with latest results.

@linev
linev merged commit 57ba97f into root-project:master Oct 23, 2026
48 of 71 checks passed
@linev
linev deleted the qt6web_fix branch October 23, 2026 05:19
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