diff options
author | Olivier Blin <oblin@mandriva.org> | 2004-07-15 10:30:49 +0000 |
---|---|---|
committer | Olivier Blin <oblin@mandriva.org> | 2004-07-15 10:30:49 +0000 |
commit | eae84851b76b64f6ebb43252d87b95ebb5a77c5d (patch) | |
tree | 6be93228a2c551ff9fb086487030082c59a40679 | |
parent | faadbbd9848ea894b57109c44313562728b89903 (diff) | |
download | userdrake-eae84851b76b64f6ebb43252d87b95ebb5a77c5d.tar userdrake-eae84851b76b64f6ebb43252d87b95ebb5a77c5d.tar.gz userdrake-eae84851b76b64f6ebb43252d87b95ebb5a77c5d.tar.bz2 userdrake-eae84851b76b64f6ebb43252d87b95ebb5a77c5d.tar.xz userdrake-eae84851b76b64f6ebb43252d87b95ebb5a77c5d.zip |
do not crash in Delete when user is in non-existing group (fix #10242)
-rwxr-xr-x | userdrake | 10 |
1 files changed, 7 insertions, 3 deletions
@@ -531,9 +531,13 @@ sub Delete { UpdateOrDelUsersInGroup($username, 1); #Let's check out the user's primary group my $usergid = $userEnt->Gid($GetValue); - my $groupEnt = $ctx->LookupGroupById($usergid); - my $member = $groupEnt->MemberName(1, 0); - if ($groupEnt && scalar(@$member) == 0) { $groupEnt->Gid($GetValue) > 499 and $ctx->GroupDel($groupEnt) }; + my $groupEnt = $ctx->LookupGroupById($usergid); + if ($groupEnt) { + my $member = $groupEnt->MemberName(1, 0); + if (scalar(@$member) == 0 and $groupEnt->Gid($GetValue) > 499) { + $ctx->GroupDel($groupEnt) + } + } $checkhome->get_active and $ctx->CleanHome($userEnt); $checkspool->get_active and $ctx->CleanSpool($userEnt); Refresh($sysfilter, $stringsearch) |