Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix crash when hiding subwindow during popup of new subwindow #80780

Merged

Conversation

HolonProduction
Copy link
Member

Fixes: #80767

The new subwindow is refered to by its index at some points. This index has to be updated directly before usage because signals that are connected to some part of the process may change the subwindow list.

@HolonProduction HolonProduction requested a review from a team as a code owner August 19, 2023 09:26
@AThousandShips AThousandShips added this to the 4.2 milestone Aug 19, 2023
@akien-mga akien-mga requested review from Sauermann and bruvzg August 19, 2023 10:34
@akien-mga akien-mga added the cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release label Aug 19, 2023
scene/main/viewport.cpp Outdated Show resolved Hide resolved
Copy link
Contributor

@Sauermann Sauermann left a comment

Choose a reason for hiding this comment

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

Changes are looking good to me and I can confirm that they are fixing the crash.

@akien-mga akien-mga merged commit b24eb34 into godotengine:master Aug 21, 2023
@akien-mga
Copy link
Member

Thanks!

@YuriSizov YuriSizov removed the cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release label Sep 20, 2023
@YuriSizov
Copy link
Contributor

Cherry-picked for 4.1.2.

@HolonProduction HolonProduction deleted the sub-window-focus-crash branch December 9, 2023 15:15
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.

SIGILL when hiding with focus_exited signal while popping up a child window
5 participants