fs/Charset: return Error on SetFSCharset() failure

Don't abort the process, let the caller decide instead.

src/Main.cxx

@@ -551,7 +551,10 @@ static int mpd_main_after_fork(struct options options)
 	GlobalEvents::Register(GlobalEvents::SHUTDOWN, shutdown_event_emitted);
 #endif
 
-	ConfigureFS();
+	if (!ConfigureFS(error)) {
+		LogError(error);
+		return EXIT_FAILURE;
+	}
 
 	if (!glue_mapper_init(error)) {
 		LogError(error);

src/fs/Charset.cxx

@@ -21,9 +21,10 @@
 #include "Charset.hxx"
 #include "Domain.hxx"
 #include "Limits.hxx"
-#include "system/FatalError.hxx"
 #include "Log.hxx"
 #include "Traits.hxx"
+#include "util/Error.hxx"
+#include "util/Domain.hxx"
 
 #ifdef HAVE_GLIB
 #include <glib.h>
@@ -36,6 +37,8 @@
 
 #ifdef HAVE_GLIB
 
+static constexpr Domain convert_domain("convert");
+
 /**
  * Maximal number of bytes required to represent path name in UTF-8
  * (including nul-terminator).
@@ -50,29 +53,37 @@ static std::string fs_charset;
 
 gcc_pure
 static bool
-IsSupportedCharset(const char *charset)
+CheckCharset(const char *charset, Error &error)
 {
 	/* convert a space to check if the charset is valid */
-	char *test = g_convert(" ", 1, charset, "UTF-8", nullptr, nullptr, nullptr);
+	GError *error2 = nullptr;
-	if (test == nullptr)
+	char *test = g_convert(" ", 1, charset, "UTF-8", nullptr, nullptr, &error2);
+	if (test == nullptr) {
+		error.Set(convert_domain, error2->code, error2->message);
+		g_error_free(error2);
 		return false;
+	}
 
 	g_free(test);
 	return true;
 }
 
-void
+bool
-SetFSCharset(const char *charset)
+SetFSCharset(const char *charset, Error &error)
 {
 	assert(charset != nullptr);
 
-	if (!IsSupportedCharset(charset))
+	if (!CheckCharset(charset, error)) {
-		FormatFatalError("invalid filesystem charset: %s", charset);
+		error.FormatPrefix("Failed to initialize filesystem charset '%s': ",
+				   charset);
+		return false;
+	}
 
 	fs_charset = charset;
 
 	FormatDebug(path_domain,
 		    "SetFSCharset: fs charset is: %s", fs_charset.c_str());
+	return true;
 }
 
 #endif

src/fs/Charset.hxx

@@ -25,6 +25,8 @@
 
 #include <string>
 
+class Error;
+
 /**
  * Gets file system character set name.
  */
@@ -32,8 +34,8 @@ gcc_const
 const char *
 GetFSCharset();
 
-void
+bool
-SetFSCharset(const char *charset);
+SetFSCharset(const char *charset, Error &error);
 
 /**
  * Convert the path to UTF-8.

src/fs/Config.cxx

@@ -29,8 +29,8 @@
 #include <glib.h>
 #endif
 
-void
+bool
-ConfigureFS()
+ConfigureFS(Error &error)
 {
 #ifdef HAVE_GLIB
 	const char *charset = nullptr;
@@ -55,7 +55,9 @@ ConfigureFS()
 #endif
 	}
 
-	if (charset != nullptr)
+	return charset == nullptr || SetFSCharset(charset, error);
-		SetFSCharset(charset);
+#else
+	(void)error;
+	return true;
 #endif
 }

src/fs/Config.hxx

@@ -22,10 +22,12 @@
 
 #include "check.h"
 
+class Error;
+
 /**
  * Performs global one-time initialization of this class.
  */
-void
+bool
-ConfigureFS();
+ConfigureFS(Error &error);
 
 #endif