After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 552522 - memory leak
memory leak
Status: RESOLVED FIXED
Product: libgnomekbd
Classification: Core
Component: Config
unspecified
Other Linux
: Normal normal
: ---
Assigned To: libgnomekbd maintainers
Sergey V. Udaltsov
Depends on:
Blocks:
 
 
Reported: 2008-09-16 17:25 UTC by Matthias Clasen
Modified: 2008-09-16 20:02 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Matthias Clasen 2008-09-16 17:25:27 UTC 
the gnome-settings-daemon keyboard plugins shows the following leak:

==24607== 28 (8 direct, 20 indirect) bytes in 1 blocks are definitely lost in loss record 4,922 of 11,209
==24607==    at 0x4006AEE: malloc (vg_replace_malloc.c:207)
==24607==    by 0xA88873: g_malloc (gmem.c:131)
==24607==    by 0xA9F3F2: g_slice_alloc (gslice.c:824)
==24607==    by 0xA9FED5: g_slist_append (gslist.c:117)
==24607==    by 0x227219: gkbd_keyboard_config_options_add (in /usr/lib/libgnomekbd.so.3.0.0)
==24607==    by 0x227458: (within /usr/lib/libgnomekbd.so.3.0.0)
==24607==    by 0x22759C: gkbd_keyboard_config_load_from_x_current (in /usr/lib/libgnomekbd.so.3.0.0)
==24607==    by 0x55421F8: apply_xkb_settings (gsd-keyboard-xkb.c:158)

What the plugin does is


gkbd_keyboard_config_init
gkbd_keyboard_config_load_from_x_current
   which does a gkbd_keyboard_config_options_add
gkbd_keyboard_config_term

It seems to me gkbd_keyboard_config_term ought to call gkbd_keyboard_config_options_reset
Comment 1 Sergey V. Udaltsov 2008-09-16 20:01:00 UTC 
Actually, we are in the code freeze stage. But since this is a bugfix, I guess we can commit that change.
Comment 2 Sergey V. Udaltsov 2008-09-16 20:02:53 UTC 
Committed!