diff options
| author | Max Kellermann <max@duempel.org> | 2009-10-16 17:37:54 +0200 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2009-10-16 17:37:54 +0200 |
| commit | d1ba27d820dc96c41cc3814801c12392060692dc (patch) | |
| tree | 05bf562b0cdf8432d5b0a921367d431def32f82b | |
| parent | 2c908fde1b5f0c0be1c7c7d4dd4fd444c31ae341 (diff) | |
| download | mpd-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).
Diffstat (limited to '')
| -rw-r--r-- | src/update.c | 3 |
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; |