update: default to follow all symlinks

MPD 0.13 and older followed all symbolic links. Although this can be a security problem (as it has always been), 0.14 should offer the same default behaviour as 0.13.

update: default to follow all symlinks
a25b8acd · Avuton Olrich · Max Kellermann · 5b11d5a3 · a25b8acd · a25b8acd
Commit a25b8acd authored Dec 08, 2008 by Avuton Olrich Committed by Max Kellermann Dec 08, 2008
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

mpd.conf.5 doc/mpd.conf.5 +1 -1

mpdconf.example doc/mpdconf.example +1 -1

update.c src/update.c +1 -1

No files found.
--- a/doc/mpd.conf.5
+++ b/doc/mpd.conf.5
@@ -38,7 +38,7 @@ This specifies the directory where saved playlists are stored.
 .B follow_outside_symlinks <yes or no>
 Control if MPD will follow symbolic links pointing outside the music dir.
 You must recreate the database after changing this option.
-The default is "no".
+The default is "yes".
 .TP
 .B follow_inside_symlinks <yes or no>
 Control if MPD will follow symbolic links pointing outside the music dir, potentially

--- a/doc/mpdconf.example
+++ b/doc/mpdconf.example
@@ -7,7 +7,7 @@ playlist_directory              "~/.mpd/playlists"
 db_file                         "~/.mpd/mpd.db"
 log_file                        "~/.mpd/mpd.log"
 error_file                      "~/.mpd/mpd.error"
-#follow_outside_symlinks         "no"
+#follow_outside_symlinks         "yes"
 #follow_inside_symlinks          "yes"
 ################################################################


--- a/src/update.c
+++ b/src/update.c
@@ -59,7 +59,7 @@ static struct condition delete_cond;

 enum {
 	DEFAULT_FOLLOW_INSIDE_SYMLINKS = true,
-	DEFAULT_FOLLOW_OUTSIDE_SYMLINKS = false,
+	DEFAULT_FOLLOW_OUTSIDE_SYMLINKS = true,
 };

 static bool follow_inside_symlinks;