From 31e7087b07b432eb6dc8ed52b8b94ee2fbfafddb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?M=C3=A1ximo=20Casta=C3=B1eda?= <antiswen@yahoo.es>
Date: Thu, 18 Apr 2024 20:06:21 +0200
Subject: [PATCH] app_server: remove new font family from the manager in case
 of error
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Change-Id: I8a1538b9706ed1c6eb5d104332ddbf3b7f223faa
Reviewed-on: https://review.haiku-os.org/c/haiku/+/7632
Haiku-Format: Haiku-format Bot <no-reply+haikuformatbot@haiku-os.org>
Reviewed-by: Jérôme Duval <jerome.duval@gmail.com>
Reviewed-by: Fredrik Holmqvist <fredrik.holmqvist@gmail.com>
Tested-by: Commit checker robot <no-reply+buildbot@haiku-os.org>
Reviewed-by: Adrien Destugues <pulkomandy@pulkomandy.tk>
---
 src/servers/app/font/FontManager.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/servers/app/font/FontManager.cpp b/src/servers/app/font/FontManager.cpp
index ff940631b9..fd241a4b35 100644
--- a/src/servers/app/font/FontManager.cpp
+++ b/src/servers/app/font/FontManager.cpp
@@ -329,8 +329,10 @@ FontManager::_AddFont(FT_Face face, node_ref nodeRef, const char* path,
 
 	if (style == NULL || !family->AddStyle(style)) {
 		delete style;
-		if (isNewFontFamily)
+		if (isNewFontFamily) {
+			fFamilies.RemoveItem(family);
 			delete family;
+		}
 		return B_NO_MEMORY;
 	}