text: font: pango/freetype2: fix correctly freeing font faces

Whoops, obvious typo. We need to free the face if the refcount drops to
zero and not if it is non-zero.

Signed-off-by: David Herrmann <dh.herrmann@googlemail.com>

diff --git a/src/text_font_freetype2.c b/src/text_font_freetype2.c
index f21d987..a3c0456 100644 (file)
--- a/src/text_font_freetype2.c
+++ b/src/text_font_freetype2.c
@@ -519,7 +519,7 @@ static void manager_put_face(struct face *face)
 {
        manager_lock();
 
-       if (--face->ref) {
+       if (!--face->ref) {
                kmscon_dlist_unlink(&face->list);
                kmscon_hashtable_free(face->glyphs);
                pthread_mutex_destroy(&face->glyph_lock);

diff --git a/src/text_font_pango.c b/src/text_font_pango.c
index 2108487..a77e584 100644 (file)
--- a/src/text_font_pango.c
+++ b/src/text_font_pango.c
@@ -343,7 +343,7 @@ static void manager_put_face(struct face *face)
 {
        manager_lock();
 
-       if (--face->ref) {
+       if (!--face->ref) {
                kmscon_dlist_unlink(&face->list);
                kmscon_hashtable_free(face->glyphs);
                pthread_mutex_destroy(&face->glyph_lock);