Stowage/godot
2
0
Fork 0
mirror of https://github.com/godotengine/godot.git synced 2025-11-03 23:21:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

Editor: Fix EditorHelpBitTooltip + ProgressDialog causes crash

Browse source
This commit is contained in:
Danil Alexeev 2024-05-08 17:39:33 +03:00
parent 107fd30ae7
commit 18fca3c294
No known key found for this signature in database
GPG key ID: 124453E157DA8DC7
2 changed files with 19 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

16
editor/editor_help.cpp
View file

@ -3760,6 +3760,14 @@ EditorHelpBit::EditorHelpBit(const String &p_symbol) {
/// EditorHelpBitTooltip /// /// EditorHelpBitTooltip ///
void EditorHelpBitTooltip::_safe_queue_free() {
if (_pushing_input > 0) {
_need_free = true;
} else {
queue_free();
}
}
void EditorHelpBitTooltip::_notification(int p_what) { void EditorHelpBitTooltip::_notification(int p_what) {
switch (p_what) { switch (p_what) {
case NOTIFICATION_WM_MOUSE_ENTER: case NOTIFICATION_WM_MOUSE_ENTER:
@ -3778,7 +3786,13 @@ void EditorHelpBitTooltip::_input_from_window(const Ref<InputEvent> &p_event) {
} else { } else {
const Ref<InputEventMouse> mouse_event = p_event; const Ref<InputEventMouse> mouse_event = p_event;
if (mouse_event.is_null()) { if (mouse_event.is_null()) {
// GH-91652. Prevents use-after-free since `ProgressDialog` calls `Main::iteration()`.
_pushing_input++;
get_parent_viewport()->push_input(p_event); get_parent_viewport()->push_input(p_event);
_pushing_input--;
if (_pushing_input <= 0 && _need_free) {
queue_free();
}
} }
} }
} }
@ -3839,7 +3853,7 @@ EditorHelpBitTooltip::EditorHelpBitTooltip(Control *p_target) {
timer = memnew(Timer); timer = memnew(Timer);
timer->set_wait_time(0.2); timer->set_wait_time(0.2);
timer->connect("timeout", callable_mp(static_cast<Node *>(this), &Node::queue_free)); timer->connect("timeout", callable_mp(this, &EditorHelpBitTooltip::_safe_queue_free));
add_child(timer); add_child(timer);
ERR_FAIL_NULL(p_target); ERR_FAIL_NULL(p_target);

4
editor/editor_help.h
View file

@ -325,6 +325,10 @@ class EditorHelpBitTooltip : public PopupPanel {
GDCLASS(EditorHelpBitTooltip, PopupPanel); GDCLASS(EditorHelpBitTooltip, PopupPanel);
Timer *timer = nullptr; Timer *timer = nullptr;
int _pushing_input = 0;
bool _need_free = false;
void _safe_queue_free();
protected: protected:
void _notification(int p_what); void _notification(int p_what);