From efb934c07fb6e6e09ebc6ec21724a3c70ca44ce3 Mon Sep 17 00:00:00 2001 From: Michael Catanzaro Date: Sun, 24 Jun 2018 15:49:46 -0500 Subject: [PATCH] imwayland: Fix a small leak If the parent get_preedit_string implementation returns a nonnull zero-length string, then we ignore it, which is almost fine. We have to free it, though. Fixes #1174 --- modules/input/imwayland.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/modules/input/imwayland.c b/modules/input/imwayland.c index bc710450d5..7e781cb987 100644 --- a/modules/input/imwayland.c +++ b/modules/input/imwayland.c @@ -491,8 +491,13 @@ gtk_im_context_wayland_get_preedit_string (GtkIMContext *context, GTK_IM_CONTEXT_CLASS (parent_class)->get_preedit_string (context, str, attrs, cursor_pos); /* If the parent implementation returns a len>0 string, go with it */ - if (str && *str && **str) - return; + if (str && *str) + { + if (**str) + return; + + g_free (*str); + } preedit_str = context_wayland->preedit.text ? context_wayland->preedit.text : "";