Merge branch 'v0.17.x'

6 files changed, 85 insertions, 15 deletions

diff --git a/.gitignore b/.gitignore
index 67b26103..9f4d606a 100644
--- a/.gitignore
+++ b/.gitignore
@@ -67,3 +67,7 @@ test/run_ntp_server
 test/run_resolver
 test/run_tcp_connect
 test/test_pcm
+test/dump_rva2
+test/dump_text_file
+test/test_byte_reverse
+test/test_vorbis_encoder
diff --git a/NEWS b/NEWS
index 0f1e2232..87f7afe5 100644
--- a/NEWS
+++ b/NEWS
@@ -21,6 +21,9 @@ ver 0.17.3 (2012/??/??)
 * decoder:
   - ffmpeg: ignore negative time stamps
   - ffmpeg: support planar audio
+* playlist:
+  - cue: fix memory leak
+  - cue: fix CUE files with only one track
 
 ver 0.17.2 (2012/09/30)
 * protocol:
diff --git a/m4/mpd_auto.m4 b/m4/mpd_auto.m4
index 23713d5b..122c2588 100644
--- a/m4/mpd_auto.m4
+++ b/m4/mpd_auto.m4
@@ -73,7 +73,8 @@ AC_DEFUN([MPD_AUTO_PKG_LIB], [
 			[eval "found_$1=yes"],
 			AC_CHECK_LIB($4, $5,
 				[eval "found_$1=yes $2_LIBS='$6' $2_CFLAGS='$7'"],
-				[eval "found_$1=no"]))
+				[eval "found_$1=no"],
+				[$6]))
 	fi
 
 	MPD_AUTO_RESULT([$1], [$8], [$9])
diff --git a/mpd.service.in b/mpd.service.in
index 9e9282fa..65fffa7b 100644
--- a/mpd.service.in
+++ b/mpd.service.in
@@ -1,6 +1,6 @@
 [Unit]
 Description=Music Player Daemon
-After=sound.target
+After=network.target sound.target
 
 [Service]
 ExecStart=@prefix@/bin/mpd --no-daemon
diff --git a/src/CommandLine.cxx b/src/CommandLine.cxx
index a784c824..a588ad5b 100644
--- a/src/CommandLine.cxx
+++ b/src/CommandLine.cxx
@@ -217,12 +217,12 @@ parse_cmdline(int argc, char **argv, struct options *options,
 				if(g_file_test(system_path,
 						G_FILE_TEST_IS_REGULAR)) {
 					ret = config_read_file(system_path,error_r);
+					g_free(system_path);
 					break;
-				}
-				++i;;
+				} else
+					g_free(system_path);
+				++i;
 			}
-			g_free(system_path);
-			g_free(&system_config_dirs);
 		}
 #else /* G_OS_WIN32 */
 		char *path2;
diff --git a/src/cue/cue_parser.c b/src/cue/cue_parser.c
index 2b0733f0..9ccc3bcd 100644
--- a/src/cue/cue_parser.c
+++ b/src/cue/cue_parser.c
@@ -58,9 +58,35 @@ struct cue_parser {
 
 	char *filename;
 
-	struct song *current, *previous, *finished;
-
+	/**
+	 * The song currently being edited.
+	 */
+	struct song *current;
+
+	/**
+	 * The previous song.  It is remembered because its end_time
+	 * will be set to the current song's start time.
+	 */
+	struct song *previous;
+
+	/**
+	 * A song that is completely finished and can be returned to
+	 * the caller via cue_parser_get().
+	 */
+	struct song *finished;
+
+	/**
+	 * Set to true after previous.end_time has been updated to the
+	 * start time of the current song.
+	 */
 	bool last_updated;
+
+	/**
+	 * Tracks whether cue_parser_finish() has been called.  If
+	 * true, then all remaining (partial) results will be
+	 * delivered by cue_parser_get().
+	 */
+	bool end;
 };
 
 struct cue_parser *
@@ -73,6 +99,7 @@ cue_parser_new(void)
 	parser->current = NULL;
 	parser->previous = NULL;
 	parser->finished = NULL;
+	parser->end = false;
 	return parser;
 }
 
@@ -85,6 +112,9 @@ cue_parser_free(struct cue_parser *parser)
 	if (parser->current != NULL)
 		song_free(parser->current);
 
+	if (parser->previous != NULL)
+		song_free(parser->previous);
+
 	if (parser->finished != NULL)
 		song_free(parser->finished);
 
@@ -201,10 +231,32 @@ cue_parse_position(const char *p)
 	return minutes * 60000 + seconds * 1000 + frames * 1000 / 75;
 }
 
+/**
+ * Commit the current song.  It will be moved to "previous", so the
+ * next song may soon edit its end time (using the next song's start
+ * time).
+ */
+static void
+cue_parser_commit(struct cue_parser *parser)
+{
+	/* the caller of this library must call cue_parser_get() often
+	   enough */
+	assert(parser->finished == NULL);
+	assert(!parser->end);
+
+	if (parser->current == NULL)
+		return;
+
+	parser->finished = parser->previous;
+	parser->previous = parser->current;
+	parser->current = NULL;
+}
+
 static void
 cue_parser_feed2(struct cue_parser *parser, char *p)
 {
 	assert(parser != NULL);
+	assert(!parser->end);
 	assert(p != NULL);
 
 	const char *command = cue_next_token(&p);
@@ -235,7 +287,7 @@ cue_parser_feed2(struct cue_parser *parser, char *p)
 		else if (parser->state == TRACK)
 			cue_add_tag(parser->current->tag, TAG_TITLE, p);
 	} else if (strcmp(command, "FILE") == 0) {
-		cue_parser_finish(parser);
+		cue_parser_commit(parser);
 
 		const char *filename = cue_next_value(&p);
 		if (filename == NULL)
@@ -258,7 +310,7 @@ cue_parser_feed2(struct cue_parser *parser, char *p)
 	} else if (parser->state == IGNORE_FILE) {
 		return;
 	} else if (strcmp(command, "TRACK") == 0) {
-		cue_parser_finish(parser);
+		cue_parser_commit(parser);
 
 		const char *nr = cue_next_token(&p);
 		if (nr == NULL)
@@ -310,6 +362,7 @@ void
 cue_parser_feed(struct cue_parser *parser, const char *line)
 {
 	assert(parser != NULL);
+	assert(!parser->end);
 	assert(line != NULL);
 
 	char *allocated = g_strdup(line);
@@ -320,12 +373,12 @@ cue_parser_feed(struct cue_parser *parser, const char *line)
 void
 cue_parser_finish(struct cue_parser *parser)
 {
-	if (parser->finished != NULL)
-		song_free(parser->finished);
+	if (parser->end)
+		/* has already been called, ignore */
+		return;
 
-	parser->finished = parser->previous;
-	parser->previous = parser->current;
-	parser->current = NULL;
+	cue_parser_commit(parser);
+	parser->end = true;
 }
 
 struct song *
@@ -333,6 +386,15 @@ cue_parser_get(struct cue_parser *parser)
 {
 	assert(parser != NULL);
 
+	if (parser->finished == NULL && parser->end) {
+		/* cue_parser_finish() has been called already:
+		   deliver all remaining (partial) results */
+		assert(parser->current == NULL);
+
+		parser->finished = parser->previous;
+		parser->previous = NULL;
+	}
+
 	struct song *song = parser->finished;
 	parser->finished = NULL;
 	return song;