From 21337811f487ca6fa5572fcfedd126f91ef723a7 Mon Sep 17 00:00:00 2001
From: Jamie Cameron <jcameron@webmin.com>
Date: Sun, 16 Mar 2008 05:22:45 +0000
Subject: [PATCH] Set correct permissions when installing a module

---
 webmin/webmin-lib.pl | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/webmin/webmin-lib.pl b/webmin/webmin-lib.pl
index e4ab949ed..d4805b959 100644
--- a/webmin/webmin-lib.pl
+++ b/webmin/webmin-lib.pl
@@ -356,8 +356,18 @@ else {
 		}
 
 	# Copy appropriate config file from modules to /etc/webmin
+	local @permmods = grep { !-d "$config_directory/$_" } @newmods;
 	system("cd $root_directory ; $perl $root_directory/copyconfig.pl '$gconfig{'os_type'}/$gconfig{'real_os_type'}' '$gconfig{'os_version'}/$gconfig{'real_os_version'}' '$install_root_directory' '$config_directory' ".join(' ', @realmods));
 
+	# Set correct permissions on *new* config directory
+	if (&supports_users()) {
+		foreach my $m (@permmods) {
+			system("chown -R root $config_directory/$m");
+			system("chgrp -R bin $config_directory/$m");
+			system("chmod -R og-rw $config_directory/$m");
+			}
+		}
+
 	# Update ACL for this user so they can access the new modules
 	&grant_user_module($grant, \@grantmods);
 	}