Skip to content

M
mpd
Commit 4f807b3a authored by Max Kellermann's avatar Max Kellermann
Browse files
Options

stored_playlist: don't map files outside the database

Don't attempt to map paths which are already absolute with map_song_fs(): check with song_in_database() instead of song_is_file().
parent fdc9a4cc
Hide whitespace changes
Inline Side-by-side
Showing with 3 additions and 2 deletions
src/playlist_save.c
View file @ 4f807b3a
......@@ -29,7 +29,7 @@ playlist_print_song(FILE *file, const struct song *song)
{
char tmp1[MPD_PATH_MAX], tmp2[MPD_PATH_MAX];
if (playlist_saveAbsolutePaths && song_is_file(song)) {
if (playlist_saveAbsolutePaths && song_in_database(song)) {
const char *path = map_song_fs(song, tmp1);
if (path != NULL)
fprintf(file, "%s\n", path);
......@@ -46,7 +46,8 @@ playlist_print_uri(FILE *file, const char *uri)
char tmp[MPD_PATH_MAX];
const char *s;
if (playlist_saveAbsolutePaths && !isValidRemoteUtf8Url(uri))
if (playlist_saveAbsolutePaths && !isValidRemoteUtf8Url(uri) &&
uri[0] != '/')
s = map_directory_child_fs(db_get_root(), uri, tmp);
else
s = utf8_to_fs_charset(tmp, uri);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment