aboutsummaryrefslogtreecommitdiffstats
path: root/src
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
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
-rw-r--r--src/state_file.c19
-rw-r--r--src/state_file.h1
3 files changed, 23 insertions, 29 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)]));
diff --git a/src/state_file.c b/src/state_file.c
index 53a6a711b..7666cc134 100644
--- a/src/state_file.c
+++ b/src/state_file.c
@@ -26,7 +26,6 @@
#include <glib.h>
#include <string.h>
-#include <sys/stat.h>
#undef G_LOG_DOMAIN
#define G_LOG_DOMAIN "state_file"
@@ -74,24 +73,18 @@ void write_state_file(void)
void read_state_file(void)
{
- struct stat st;
unsigned int i;
FILE *fp;
get_state_file_path();
if (!sfpath)
return;
- if (stat(sfpath, &st) < 0) {
- g_debug("failed to stat %s: %s", sfpath, strerror(errno));
- return;
- }
- if (!S_ISREG(st.st_mode))
- g_error("\"%s\" is not a regular file", sfpath);
while (!(fp = fopen(sfpath, "r")) && errno == EINTR);
if (G_UNLIKELY(!fp)) {
- g_error("failed to open %s: %s",
- sfpath, strerror(errno));
+ g_warning("failed to open %s: %s",
+ sfpath, strerror(errno));
+ return;
}
for (i = 0; i < ARRAY_SIZE(sf_callbacks); i++) {
sf_callbacks[i].reader(fp);
@@ -100,9 +93,3 @@ void read_state_file(void)
while(fclose(fp) && errno == EINTR) /* nothing */;
}
-
-void G_GNUC_NORETURN state_file_fatal(void)
-{
- g_error("failed to parse %s", sfpath);
-}
-
diff --git a/src/state_file.h b/src/state_file.h
index 0fa1793d5..229df2861 100644
--- a/src/state_file.h
+++ b/src/state_file.h
@@ -23,6 +23,5 @@
void write_state_file(void);
void read_state_file(void);
-void G_GNUC_NORETURN state_file_fatal(void);
#endif /* STATE_FILE_H */