Add conf_mount_rw calls on functions that changes user/groups. It fixes #3294

1 files changed, 12 insertions, 0 deletions

diff --git a/etc/inc/auth.inc b/etc/inc/auth.inc
index 67fd8b1..c4a3ae8 100644
--- a/etc/inc/auth.inc
+++ b/etc/inc/auth.inc
@@ -485,6 +485,8 @@ function local_user_set(& $user) {
 function local_user_del($user) {
 	global $debug;
 
+	conf_mount_rw();
+
 	/* remove all memberships */
 	local_user_set_groups($user);
 
@@ -512,6 +514,8 @@ function local_user_del($user) {
 
 	/* Delete user from groups needs a call to write_config() */
 	local_group_del_user($user);
+
+	conf_mount_ro();
 }
 
 function local_user_set_password(& $user, $password) {
@@ -617,6 +621,8 @@ function local_group_del_user($user) {
 function local_group_set($group, $reset = false) {
 	global $debug;
 
+	conf_mount_rw();
+
 	$group_name = $group['name'];
 	$group_gid = $group['gid'];
 	$group_members = "''";
@@ -641,17 +647,23 @@ function local_group_set($group, $reset = false) {
 		log_error(sprintf(gettext("Running: %s"), $cmd));
 	mwexec($cmd);
 
+	conf_mount_ro();
+
 }
 
 function local_group_del($group) {
 	global $debug;
 
+	conf_mount_rw();
+
 	/* delete from group db */
 	$cmd = "/usr/sbin/pw groupdel {$group['name']}";
 
 	if($debug)
 		log_error(sprintf(gettext("Running: %s"), $cmd));
 	mwexec($cmd);
+
+	conf_mount_ro();
 }
 
 function ldap_test_connection($authcfg) {