From 87a8a3e22f7d25c88c344cd195bde58d237716a0 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Mon, 2 Mar 2015 21:48:37 +0100 Subject: PlaylistFile: move memchr() call to class Path --- src/PlaylistFile.cxx | 6 ++++-- src/fs/Path.hxx | 10 ++++++++++ 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/src/PlaylistFile.cxx b/src/PlaylistFile.cxx index e22cacfea..7b461e387 100644 --- a/src/PlaylistFile.cxx +++ b/src/PlaylistFile.cxx @@ -169,11 +169,13 @@ LoadPlaylistFileInfo(PlaylistInfo &info, const Path parent_path_fs, const Path name_fs) { + if (name_fs.HasNewline()) + return false; + const auto *const name_fs_str = name_fs.c_str(); const size_t name_length = name_fs.length(); - if (name_length < ARRAY_SIZE(PLAYLIST_FILE_SUFFIX) || - memchr(name_fs_str, '\n', name_length) != nullptr) + if (name_length < ARRAY_SIZE(PLAYLIST_FILE_SUFFIX)) return false; if (!StringEndsWith(name_fs_str, PLAYLIST_FILE_SUFFIX)) diff --git a/src/fs/Path.hxx b/src/fs/Path.hxx index 8126a668a..962518896 100644 --- a/src/fs/Path.hxx +++ b/src/fs/Path.hxx @@ -122,6 +122,16 @@ public: return value; } + /** + * Does the path contain a newline character? (Which is + * usually rejected by MPD because its protocol cannot + * transfer newline characters). + */ + gcc_pure + bool HasNewline() const { + return strchr(value, '\n') != nullptr; + } + /** * Convert the path to UTF-8. * Returns empty string on error or if this instance is "nulled" -- cgit v1.2.3