This is an automated email from the ASF dual-hosted git repository.

arkurth pushed a commit to branch 
VCL-1083_check_NFS_mounts_under_Linux_home_directories
in repository https://gitbox.apache.org/repos/asf/vcl.git

commit 0e1f6f73bd2b568c9c7ed66812466a14df2ca327
Author: Andy Kurth <arku...@apache.org>
AuthorDate: Mon Aug 5 13:09:42 2019 -0400

    VCL-1083
    Fixed logic problem in Linux.pm::delete_user introduced in previous commit 
to prevent home directories on non-local disks from being deleted.
---
 managementnode/lib/VCL/Module/OS/Linux.pm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/managementnode/lib/VCL/Module/OS/Linux.pm 
b/managementnode/lib/VCL/Module/OS/Linux.pm
index dda70ab..8ddc684 100644
--- a/managementnode/lib/VCL/Module/OS/Linux.pm
+++ b/managementnode/lib/VCL/Module/OS/Linux.pm
@@ -3097,9 +3097,11 @@ sub delete_user {
        # Assemble the userdel command
        my $userdel_command = "/usr/sbin/userdel";
        
-       my $delete_home_directory = 1;
+       my $delete_home_directory = 0;
        
        if ($home_directory_on_local_disk) {
+               $delete_home_directory = 1;
+               
                # Fetch exclude_list
                my @exclude_list = $self->get_exclude_list();
                if ((grep(/\/home\/$username/, @exclude_list))) {

Reply via email to