diff options
author | David Edmundson <davidedmundson@kde.org> | 2023-12-15 08:57:47 +0000 |
---|---|---|
committer | Liang Qi <liang.qi@qt.io> | 2023-12-21 12:17:49 +0100 |
commit | 18166784fda5b3afce55bf00227f77e08ff08ac2 (patch) | |
tree | 26ba3544ddd4041ca6f472552014b2ab025f1f4f | |
parent | d00b67096ac98e2a5e3ea924673f72acaab41e83 (diff) |
client: Clean up text input proxy object in textinputv2
This not only cleans up a tiny amount of memory, but also fixes a
potential crash if a wayland event were to arrive on the object after
the Qt level wrapper is destroyed.
Pick-to: 6.5 6.6 6.7
Change-Id: I6fafdd848d61da893304c4467a57b52d3771c508
Reviewed-by: Liang Qi <liang.qi@qt.io>
-rw-r--r-- | src/client/qwaylandtextinputv2.cpp | 1 | ||||
-rw-r--r-- | tests/auto/client/shared/textinput.cpp | 3 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/client/qwaylandtextinputv2.cpp b/src/client/qwaylandtextinputv2.cpp index a799793e5..89a8a6d77 100644 --- a/src/client/qwaylandtextinputv2.cpp +++ b/src/client/qwaylandtextinputv2.cpp @@ -46,6 +46,7 @@ QWaylandTextInputv2::~QWaylandTextInputv2() { if (m_resetCallback) wl_callback_destroy(m_resetCallback); + destroy(); } void QWaylandTextInputv2::reset() diff --git a/tests/auto/client/shared/textinput.cpp b/tests/auto/client/shared/textinput.cpp index 05889687c..ee7e9ff9b 100644 --- a/tests/auto/client/shared/textinput.cpp +++ b/tests/auto/client/shared/textinput.cpp @@ -12,9 +12,8 @@ TextInputManager::TextInputManager(CoreCompositor *compositor) void TextInputManager::zwp_text_input_manager_v2_get_text_input(Resource *resource, uint32_t id, wl_resource *seatResource) { - Q_UNUSED(resource); - Q_UNUSED(id); Q_UNUSED(seatResource); + add(resource->client(), id, resource->version()); } } // namespace MockCompositor |