VladPolskiy/webmin
1
0
Fork 0
mirror of https://github.com/webmin/webmin.git synced 2025-08-17 19:06:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix setting of second groups for LDAP users

Browse Source 
https://sourceforge.net/tracker/?func=detail&atid=117457&aid=3412546&group_id=17457
This commit is contained in:
Jamie Cameron
2011-09-21 13:23:58 -07:00
parent 11bef5910a
commit dc2df93f90
1 changed files with 10 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
ldap-useradmin/save_user.cgi
View File

@ -621,11 +621,13 @@ else {
foreach $g ($rv->all_entries) {
local @mems = $g->get_value("memberUid");
local $gname = $g->get_value("cn");
local $ldap_group_id = $g->get_value("gidNumber");
if ($renaming) {
local $idx = &indexof($olduser, @mems);
if ($ingroup{$gname} && $idx<0) {
# Need to add to the group
push(@mems, $user);
push(@sgids, $ldap_group_id);
}
elsif (!$ingroup{$gname} && $idx>=0) {
# Need to remove from the group
@ -634,6 +636,7 @@ else {
elsif ($idx >= 0) {
# Need to rename in group
$mems[$idx] = $user;
push(@sgids, $ldap_group_id);
}
else { next; }
}
@ -642,11 +645,17 @@ else {
if ($ingroup{$gname} && $idx<0) {
# Need to add to the group
push(@mems, $user);
push(@sgids, $ldap_group_id);
}
elsif (!$ingroup{$gname} && $idx>=0) {
# Need to remove from the group
splice(@mems, $idx, 1);
}
elsif ($ingroup{$gname} && $idx >=0) {
# already in this group
push(@sgids, $ldap_group_id);
next;
}
else { next; }
}
@ -669,7 +678,7 @@ else {
# Run post-change script
&set_user_envs(\%user, $in{'new'} ? 'CREATE_USER' : 'MODIFY_USER',
$in{'passmode'} == 3 ? $in{'pass'} : "", undef);
$in{'passmode'} == 3 ? $in{'pass'} : "", \@sgids);
&made_changes();
# Run other modules' scripts