! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Memory leak in keyboard shortcut code
Status:
RESOLVED: FIXED
Product:
Libxfce4ui
Component:
General

Comments

Description Olivier Fourdan editbugs 2011-10-28 16:37:31 CEST
Created attachment 3923 
Fix a memleak in xfce_shortcuts_provider_get_shortcuts()

Valgrind gives:

==31203== 2,252 bytes in 52 blocks are definitely lost in loss record 2,559 of 2,652
==31203==    at 0x4C2649D: malloc (vg_replace_malloc.c:236)
==31203==    by 0x8C90600: g_malloc (gmem.c:164)
==31203==    by 0x8CA71F4: g_strconcat (gstrfuncs.c:327)
==31203==    by 0x5463F4B: ??? (in /usr/lib64/libxfce4kbd-private-2.so.0.0.0)
==31203==    by 0x8C79152: g_hash_table_foreach (ghash.c:1326)
==31203==    by 0x5464D5B: xfce_shortcuts_provider_get_shortcuts (in /usr/lib64/libxfce4kbd-private-2.so.0.0.0)
==31203==    by 0x42EBB0: loadSettings (settings.c:571)
==31203==    by 0x42FA83: reloadScreenSettings (settings.c:954)
==31203==    by 0x83F834D: g_closure_invoke (gclosure.c:767)
==31203==    by 0x84091FF: signal_emit_unlocked_R (gsignal.c:3252)
==31203==    by 0x84129D1: g_signal_emit_valist (gsignal.c:2983)
==31203==    by 0x8412B71: g_signal_emit (gsignal.c:3040)

Patch attached - Please apply in both branches, master and xfce-4.8
Comment 1 Nick Schermer editbugs 2011-10-28 17:46:48 CEST
Indeed, pushed fix.

Bug #8079

Reported by:
Olivier Fourdan
Reported on: 2011-10-28
Last modified on: 2011-10-28

People

Assignee:
Nick Schermer
CC List:
1 user

Version

Attachments

Additional information