summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormkanat%bugzilla.org <>2006-08-08 08:06:51 +0200
committermkanat%bugzilla.org <>2006-08-08 08:06:51 +0200
commitab61cd7a33c69493babc445b3b8a8bbd8b7c08f5 (patch)
tree5d515207741ef30e721bc5142dba06acc22d3e64
parentd12c283343406c9a5c4b5e76c1f77dce354efc38 (diff)
downloadbugzilla-ab61cd7a33c69493babc445b3b8a8bbd8b7c08f5.tar.gz
bugzilla-ab61cd7a33c69493babc445b3b8a8bbd8b7c08f5.tar.xz
Bug 347706: Params can't be updated when webservergroup is empty
Patch By Max Kanat-Alexander <mkanat@bugzilla.org> (module owner) a=myk
-rw-r--r--Bugzilla/Install/Filesystem.pm7
1 files changed, 5 insertions, 2 deletions
diff --git a/Bugzilla/Install/Filesystem.pm b/Bugzilla/Install/Filesystem.pm
index 893870e6d..76116bdff 100644
--- a/Bugzilla/Install/Filesystem.pm
+++ b/Bugzilla/Install/Filesystem.pm
@@ -82,6 +82,9 @@ sub FILESYSTEM {
my $owner_dir_readable = 0700;
# Writeable by the web server.
my $ws_dir_writeable = $ws_group ? 0770 : 01777;
+ # The webserver can overwrite files owned by other users,
+ # in this directory.
+ my $ws_dir_full_control = $ws_group ? 0770 : 0777;
# Note: When being processed by checksetup, these have their permissions
# set in this order: %all_dirs, %recurse_dirs, %all_files.
@@ -126,7 +129,7 @@ sub FILESYSTEM {
my %recurse_dirs = (
# Writeable directories
"$datadir/template" => { files => $ws_readable,
- dirs => $ws_dir_writeable },
+ dirs => $ws_dir_full_control },
$attachdir => { files => $ws_writeable,
dirs => $ws_dir_writeable },
$webdotdir => { files => $ws_writeable,
@@ -170,7 +173,7 @@ sub FILESYSTEM {
# The name of each directory that we should actually *create*,
# pointing at its default permissions.
my %create_dirs = (
- $datadir => $ws_dir_writeable,
+ $datadir => $ws_dir_full_control,
"$datadir/mimedump-tmp" => $ws_dir_writeable,
"$datadir/mining" => $ws_dir_readable,
"$datadir/duplicates" => $ws_dir_readable,