Page MenuHomeFreeBSD

x11/libxfce4menu: Update fix for leaked keygrabs when layout changes
ClosedPublic

Authored by madpilot on May 25 2020, 8:30 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 20, 5:42 PM
Unknown Object (File)
Fri, Apr 12, 9:48 AM
Unknown Object (File)
Fri, Apr 12, 6:04 AM
Unknown Object (File)
Thu, Apr 11, 6:41 PM
Unknown Object (File)
Mar 9 2024, 3:52 PM
Unknown Object (File)
Feb 17 2024, 11:32 PM
Unknown Object (File)
Feb 17 2024, 7:50 AM
Unknown Object (File)
Feb 7 2024, 10:23 PM
Subscribers

Details

Summary

Update to the fix proposed by cem in [1].

This patch uses the glib GArray API to have growing list of old keycodes instead of using a fixed size array.

The patch is submitted upstream at [2].

[1] https://reviews.freebsd.org/D24338

[2] https://gitlab.xfce.org/xfce/libxfce4ui/-/merge_requests/1

Test Plan

I'm runing this on my machine without problems.

I'd like someone else to confirm it works properly.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 31292
Build 28933: arc lint + arc unit

Event Timeline

x11/libxfce4menu/files/patch-libxfce4kbd-private_xfce-shortcuts-grabber.c
17–18

I suspect this check is not really necessary anymore, but not completely sure.

I also was not able to find documentation about the behavior of g_array_index() when one asks to access an element beyond the end of the array.

Align patch with the one sent upstream.

madpilot accepted this revision.
This revision is now accepted and ready to land.Jul 15 2020, 1:24 PM