diff options
| author | Colomban Wendling <[email protected]> | 2023-06-27 12:21:16 +0200 |
|---|---|---|
| committer | raveit65 <[email protected]> | 2024-01-25 18:52:24 +0100 |
| commit | fa8a4d4c2f029c6586d2f27384114e2c2e2e79b8 (patch) | |
| tree | a821df3beba69d8072c763c0bf1f8a01551f8cc5 /plugins/mpris | |
| parent | 9c70692c3630768f6b45e890e45b3e72a536a4f2 (diff) | |
| download | mate-settings-daemon-fa8a4d4c2f029c6586d2f27384114e2c2e2e79b8.tar.bz2 mate-settings-daemon-fa8a4d4c2f029c6586d2f27384114e2c2e2e79b8.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/mpris')
0 files changed, 0 insertions, 0 deletions