aboutsummaryrefslogtreecommitdiffstats
path: root/src/playlist.c
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2009-01-03 14:53:23 +0100
committerMax Kellermann <max@duempel.org>2009-01-03 14:53:23 +0100
commitdcff29e5aa18d8957635553753c4b4ddd730d790 (patch)
tree903e2a67e665d777134b53cb5443a3e5505c41d0 /src/playlist.c
parent2064e8ac4cb4c7b7c69042e6dc1715c18777b983 (diff)
downloadmpd-dcff29e5aa18d8957635553753c4b4ddd730d790.tar.gz
mpd-dcff29e5aa18d8957635553753c4b4ddd730d790.tar.xz
mpd-dcff29e5aa18d8957635553753c4b4ddd730d790.zip
state_file: errors are non-fatal in read_state_file()
If the state file cannot be read, for whatever reason, don't abort MPD. The state file isn't _that_ important.
Diffstat (limited to '')
-rw-r--r--src/playlist.c32
1 files changed, 20 insertions, 12 deletions
diff --git a/src/playlist.c b/src/playlist.c
index c36bb4bbe..49b3460f0 100644
--- a/src/playlist.c
+++ b/src/playlist.c
@@ -30,7 +30,6 @@
#include "log.h"
#include "mapper.h"
#include "path.h"
-#include "state_file.h"
#include "stored_playlist.h"
#include "ack.h"
#include "idle.h"
@@ -279,17 +278,26 @@ static void loadPlaylistFromStateFile(FILE *fp, char *buffer,
char *temp;
int song;
- if (!fgets(buffer, PLAYLIST_BUFFER_SIZE, fp))
- state_file_fatal();
+ if (!fgets(buffer, PLAYLIST_BUFFER_SIZE, fp)) {
+ g_warning("No playlist in state file");
+ return;
+ }
+
while (!g_str_has_prefix(buffer, PLAYLIST_STATE_FILE_PLAYLIST_END)) {
g_strchomp(buffer);
temp = strtok(buffer, ":");
- if (temp == NULL)
- state_file_fatal();
+ if (temp == NULL) {
+ g_warning("Malformed playlist line in state file");
+ break;
+ }
+
song = atoi(temp);
- if (!(temp = strtok(NULL, "")))
- state_file_fatal();
+ if (!(temp = strtok(NULL, ""))) {
+ g_warning("Malformed playlist line in state file");
+ break;
+ }
+
if (addToPlaylist(temp, NULL) == PLAYLIST_RESULT_SUCCESS
&& current == song) {
if (state != PLAYER_STATE_STOP) {
@@ -304,8 +312,11 @@ static void loadPlaylistFromStateFile(FILE *fp, char *buffer,
}
}
- if (!fgets(buffer, PLAYLIST_BUFFER_SIZE, fp))
- state_file_fatal();
+ if (!fgets(buffer, PLAYLIST_BUFFER_SIZE, fp)) {
+ g_warning("'%s' not found in state file",
+ PLAYLIST_STATE_FILE_PLAYLIST_END);
+ break;
+ }
}
}
@@ -357,9 +368,6 @@ void readPlaylistState(FILE *fp)
} else
setPlaylistRandomStatus(false);
} else if (g_str_has_prefix(buffer, PLAYLIST_STATE_FILE_CURRENT)) {
- if (strlen(buffer) ==
- strlen(PLAYLIST_STATE_FILE_CURRENT))
- state_file_fatal();
current = atoi(&(buffer
[strlen
(PLAYLIST_STATE_FILE_CURRENT)]));