diff options
author | Colomban Wendling <[email protected]> | 2023-06-27 12:21:16 +0200 |
---|---|---|
committer | raveit65 <[email protected]> | 2023-06-29 18:07:38 +0200 |
commit | 3f0a8070731e8e15f75e91dad715e479eae8a7b3 (patch) | |
tree | 23b141f5498a7d7ab4fb770dc192ebf82262e3ec /plugins/a11y-keyboard/msd-a11y-keyboard-atspi.h | |
parent | 3a216ce3d8ad81ed31f447e39c380e9b1df0701a (diff) | |
download | mate-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.h')
0 files changed, 0 insertions, 0 deletions