From 15d4c841ce8b5b387850d710999c623b038910fa Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Tue, 30 Jun 2009 16:29:40 +0200 Subject: database: fixed NULL pointer dereference after charset change When the filesystem_charset is changed in mpd.conf, MPD should discard the old database. In this error branch, MPD did not fill the GError object properly, and logged a warning message instead, which caused a segmentation fault. --- src/database.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'src') diff --git a/src/database.c b/src/database.c index 7257930b..5a06dda9 100644 --- a/src/database.c +++ b/src/database.c @@ -318,10 +318,11 @@ db_load(GError **error) if (old_charset != NULL && strcmp(new_charset, old_charset)) { fclose(fp); - g_message("Existing database has charset \"%s\" " - "instead of \"%s\"; " - "discarding database file", - new_charset, old_charset); + g_set_error(error, db_quark(), 0, + "Existing database has charset " + "\"%s\" instead of \"%s\"; " + "discarding database file", + new_charset, old_charset); return false; } } else { -- cgit v1.2.3