summaryrefslogtreecommitdiff
path: root/plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c
diff options
context:
space:
mode:
authorColomban Wendling <[email protected]>2023-06-27 12:21:16 +0200
committerraveit65 <[email protected]>2023-06-29 18:07:38 +0200
commit3f0a8070731e8e15f75e91dad715e479eae8a7b3 (patch)
tree23b141f5498a7d7ab4fb770dc192ebf82262e3ec /plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c
parent3a216ce3d8ad81ed31f447e39c380e9b1df0701a (diff)
downloadmate-settings-daemon-3f0a8070731e8e15f75e91dad715e479eae8a7b3.tar.bz2
mate-settings-daemon-3f0a8070731e8e15f75e91dad715e479eae8a7b3.tar.xz
a11y-keyboard-atspi: Fix memory leak
Fix fairly large memory leak when beeping on keys while caps lock is enabled. The libatspi2 docs and API were quite misleading, so I overlooked the fact the event parameter should be freed in the callback. This changes the constness of the callback argument, which is new in libatspi2 2.40 -- yet the actual behavior didn't change, only the qualifier was removed, see [1]. This might however bring up a compiler warning when building against libatspi2 < 2.40; but on the other hand it fixed build with clang >= 16, see #399. As it is unlikely to build with clang >= 16 and libatspi2 < 2.40, I think it's a good compromise. [1] https://gitlab.gnome.org/GNOME/at-spi2-core/-/commit/7dfb0b7fc2d1710ef7fad54f910fa4c6a5e3af17
Diffstat (limited to 'plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c')
-rw-r--r--plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c b/plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c
index ecdacb8..a45ef47 100644
--- a/plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c
+++ b/plugins/a11y-keyboard/msd-a11y-keyboard-atspi.c
@@ -55,16 +55,16 @@ msd_a11y_keyboard_atspi_class_init (MsdA11yKeyboardAtspiClass *klass)
}
static gboolean
-on_key_press_event (const AtspiDeviceEvent *event,
- void *user_data G_GNUC_UNUSED)
+on_key_press_event (AtspiDeviceEvent *event,
+ void *user_data G_GNUC_UNUSED)
{
/* don't ring on capslock itself, that's taken care of by togglekeys
* if the user want it. */
- if (event->id == GDK_KEY_Caps_Lock)
- return FALSE;
-
- gdk_display_beep (gdk_display_get_default ());
+ if (event->id != GDK_KEY_Caps_Lock)
+ gdk_display_beep (gdk_display_get_default ());
+ /* cast the possible erroneous const away with atspi < 2.40 */
+ g_boxed_free (ATSPI_TYPE_DEVICE_EVENT, (gpointer) event);
return FALSE;
}