From eda9e9303d866e299c9bc03ea041552d4a62c85c Mon Sep 17 00:00:00 2001 From: Marco Peereboom Date: Wed, 7 Jul 2010 12:52:42 +0000 Subject: [PATCH] Don't leak font memory when setting up the bar a subsequent time --- scrotwm.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/scrotwm.c b/scrotwm.c index 49dec92..d04ecca 100644 --- a/scrotwm.c +++ b/scrotwm.c @@ -922,6 +922,11 @@ bar_setup(struct swm_region *r) { int i; + if (bar_fs) { + XFreeFont(display, bar_fs); + bar_fs = NULL; + } + for (i = 0; bar_fonts[i] != NULL; i++) { bar_fs = XLoadQueryFont(display, bar_fonts[i]); if (bar_fs) { @@ -931,6 +936,9 @@ bar_setup(struct swm_region *r) } if (bar_fonts[i] == NULL) errx(1, "couldn't load font"); + if (bar_fs == NULL) + errx(1, "couldn't create font structure"); + bar_height = bar_fs->ascent + bar_fs->descent + 3; r->bar_window = XCreateSimpleWindow(display, -- 1.7.10.4