From b84bf3585ce68784406ebe435dca5f67406db66e Mon Sep 17 00:00:00 2001 From: emersion Date: Sun, 25 Nov 2018 09:09:32 +0100 Subject: [PATCH] Fix pango title escaping This commit fixes two bugs. First, commit [1] has inverted the condition when we escape pango markup. We need to escape client-provided strings when markup is enabled. Second, parse_title_format has a shortcut when title_format is set to `%title`, and escape_pango_markup wasn't used anymore there. Fixes https://github.com/swaywm/sway/issues/3181 [1]: https://github.com/swaywm/sway/pull/3179/commits/caee2dff03fc007dc46cf121e013f5347ac46ba9 --- sway/tree/view.c | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/sway/tree/view.c b/sway/tree/view.c index 63bb8e26d..febba3b9f 100644 --- a/sway/tree/view.c +++ b/sway/tree/view.c @@ -835,11 +835,10 @@ static size_t append_prop(char *buffer, const char *value) { if (!value) { return 0; } - // if using pango_markup in font, we need to escape all markup char - // from values to avoid messing with pango markup - if (!config->pango_markup) { + // If using pango_markup in font, we need to escape all markup chars + // from values to make sure tags are not inserted by clients + if (config->pango_markup) { char *escaped_value = escape_pango_markup(value); - lenient_strcat(buffer, escaped_value); size_t len = strlen(escaped_value); free(escaped_value); @@ -856,11 +855,7 @@ static size_t append_prop(char *buffer, const char *value) { */ static size_t parse_title_format(struct sway_view *view, char *buffer) { if (!view->title_format || strcmp(view->title_format, "%title") == 0) { - const char *title = view_get_title(view); - if (buffer && title) { - strcpy(buffer, title); - } - return title ? strlen(title) : 0; + return append_prop(buffer, view_get_title(view)); } size_t len = 0;