diff options
author | Ray Strode <rstrode@redhat.com> | 2012-03-21 16:51:28 -0400 |
---|---|---|
committer | Sergey V. Udaltsov <svu@gnome.org> | 2012-03-22 09:23:00 +0000 |
commit | a6ca20e027edf43df578c7cf1aa26eecb6512540 (patch) | |
tree | 2ad5459c8dee95e3af1840d0d070fa658e3d2001 | |
parent | d63547c01abff6a24920445455c8d2ede2eadd02 (diff) | |
download | libgnomekbd-a6ca20e027edf43df578c7cf1aa26eecb6512540.tar.gz |
config: plug minor leak
gkbd_keyboard_config_split_items allocates its out args, but
those allocations aren't getting cleaned up in
gkbd_keyboard_config_copy_to_xkl_config
This commit plugs that leak.
https://bugzilla.gnome.org/show_bug.cgi?id=670164
-rw-r--r-- | libgnomekbd/gkbd-keyboard-config.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/libgnomekbd/gkbd-keyboard-config.c b/libgnomekbd/gkbd-keyboard-config.c index acd5456..397df24 100644 --- a/libgnomekbd/gkbd-keyboard-config.c +++ b/libgnomekbd/gkbd-keyboard-config.c @@ -309,9 +309,11 @@ gkbd_keyboard_config_copy_to_xkl_config (GkbdKeyboardConfig * kbd_config, char *group, *option; if (gkbd_keyboard_config_split_items (*the_option, &group, &option) - && option != NULL) + && option != NULL) { *(p++) = g_strdup (option); - else { + g_free (group); + g_free (option); + } else { *(p++) = g_strdup (""); xkl_debug (150, "Could not split [%s]\n", *the_option); |