introduced state to differentiate between enabled/disabled inheritance. renamed copy to reorganize, since the meaning for inheritance=enabled is different

This commit is contained in:
Hans-Joachim Kliemeck 2015-10-21 22:43:42 +02:00 committed by Matt Clay
parent bdf0a888bb
commit 632df80276
2 changed files with 59 additions and 18 deletions

View file

@ -26,7 +26,8 @@ $result = New-Object PSObject;
Set-Attr $result "changed" $false;
$path = Get-Attr $params "path" -failifempty $true
$copy = Get-Attr $params "copy" "no" -validateSet "no","yes" -resultobj $result | ConvertTo-Bool
$state = Get-Attr $params "state" "absent" -validateSet "present","absent" -resultobj $result
$reorganize = Get-Attr $params "reorganize" "no" -validateSet "no","yes" -resultobj $result | ConvertTo-Bool
If (-Not (Test-Path -Path $path)) {
Fail-Json $result "$path file or directory does not exist on the host"
@ -34,19 +35,44 @@ If (-Not (Test-Path -Path $path)) {
Try {
$objACL = Get-ACL $path
$alreadyDisabled = !$objACL.AreAccessRulesProtected
$inheritanceEnabled = !$objACL.AreAccessRulesProtected
If ($copy) {
$objACL.SetAccessRuleProtection($True, $True)
} Else {
$objACL.SetAccessRuleProtection($True, $False)
}
If (($state -eq "present") -And !$inheritanceEnabled) {
If ($reorganize) {
$objACL.SetAccessRuleProtection($True, $True)
} Else {
$objACL.SetAccessRuleProtection($True, $False)
}
If ($alreadyDisabled) {
Set-ACL $path $objACL
Set-Attr $result "changed" $true;
}
Elseif (($state -eq "absent") -And $inheritanceEnabled) {
# second parameter is ignored if first=$False
$objACL.SetAccessRuleProtection($False, $False)
Set-ACL $path $objACL
If ($reorganize) {
# convert explicit ACE to inherited ACE
ForEach($inheritedRule in $objACL.Access) {
If (!$inheritedRule.IsInherited) {
Continue
}
ForEach($explicitRrule in $objACL.Access) {
If ($inheritedRule.IsInherited) {
Continue
}
If (($inheritedRule.FileSystemRights -eq $explicitRrule.FileSystemRights) -And ($inheritedRule.AccessControlType -eq $explicitRrule.AccessControlType) -And ($inheritedRule.IdentityReference -eq $explicitRrule.IdentityReference) -And ($inheritedRule.InheritanceFlags -eq $explicitRrule.InheritanceFlags) -And ($inheritedRule.PropagationFlags -eq $explicitRrule.PropagationFlags)) {
$objACL.RemoveAccessRule($explicitRrule)
}
}
}
}
Set-ACL $path $objACL
Set-Attr $result "changed" $true;
}
}
Catch {
Fail-Json $result "an error occured when attempting to disable inheritance"

View file

@ -25,17 +25,25 @@ DOCUMENTATION = '''
---
module: win_acl_inheritance
version_added: "2.0"
short_description: Disable ACL inheritance
short_description: Change ACL inheritance
description:
- Disable ACL (Access Control List) inheritance and optionally converts ACE (Access Control Entry) to dedicated ACE
- Change ACL (Access Control List) inheritance and optionally copy inherited ACE's (Access Control Entry) to dedicated ACE's or vice versa.
options:
path:
description:
- Path to be used for disabling
- Path to be used for changing inheritance
required: true
copy:
state:
description:
- Indicates if the inherited ACE should be copied to dedicated ACE
- Specify whether to enable I(present) or disable I(absent) ACL inheritance
required: false
choices:
- present
- absent
default: absent
reorganize:
description:
- For P(state) = I(absent), indicates if the inherited ACE's should be copied. For P(state) = I(present), indicates if the inherited ACE's should be simplified.
required: false
choices:
- no
@ -47,13 +55,20 @@ author: Hans-Joachim Kliemeck (@h0nIg)
EXAMPLES = '''
# Playbook example
---
- name: Disable and copy
- name: Disable inherited ACE's
win_acl_inheritance:
path: 'C:\\apache\\'
copy: yes
state: absent
- name: Disable
- name: Disable and copy inherited ACE's
win_acl_inheritance:
path: 'C:\\apache\\'
copy: no
state: absent
reorganize: yes
- name: Enable and remove dedicated ACE's
win_acl_inheritance:
path: 'C:\\apache\\'
state: present
reorganize: yes
'''