aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2009-10-16 17:37:54 +0200
committerMax Kellermann <max@duempel.org>2009-10-16 17:37:54 +0200
commitd1ba27d820dc96c41cc3814801c12392060692dc (patch)
tree05bf562b0cdf8432d5b0a921367d431def32f82b
parent2c908fde1b5f0c0be1c7c7d4dd4fd444c31ae341 (diff)
downloadmpd-d1ba27d820dc96c41cc3814801c12392060692dc.tar.gz
mpd-d1ba27d820dc96c41cc3814801c12392060692dc.tar.xz
mpd-d1ba27d820dc96c41cc3814801c12392060692dc.zip
update: song_file_new() cannot fail
Removed the NULL check. If that NULL check was correct, that would have been a memory leak (vtrack).
-rw-r--r--src/update.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/src/update.c b/src/update.c
index 593198cb9..4aa48f25c 100644
--- a/src/update.c
+++ b/src/update.c
@@ -459,8 +459,6 @@ update_container_file( struct directory* directory,
while ((vtrack = plugin->container_scan(pathname, ++tnum)) != NULL)
{
struct song* song = song_file_new(vtrack, contdir);
- if (song == NULL)
- return true;
// shouldn't be necessary but it's there..
song->mtime = st->st_mtime;
@@ -468,7 +466,6 @@ update_container_file( struct directory* directory,
song->tag = plugin->tag_dup(map_directory_child_fs(contdir, vtrack));
songvec_add(&contdir->songs, song);
- song = NULL;
modified = true;