Merge pull request #35054 from lubyou/fix-win_dacl-disable_inheritance

Only fail user lookup is the user parameter is required
This commit is contained in:
Mike Place 2016-07-29 07:58:41 -06:00 committed by GitHub
commit f34bb7a8de

View file

@ -347,8 +347,7 @@ def get(path, objectType, user=None):
'ACLs': []}
sidRet = _getUserSid(user)
if not sidRet['result']:
return sidRet
if path and objectType:
dc = daclConstants()
objectTypeBit = dc.getObjectTypeBit(objectType)
@ -645,8 +644,6 @@ def check_inheritance(path, objectType, user=None):
'comment': ''}
sidRet = _getUserSid(user)
if not sidRet['result']:
return sidRet
dc = daclConstants()
objectType = dc.getObjectTypeBit(objectType)
@ -665,7 +662,8 @@ def check_inheritance(path, objectType, user=None):
if (ace[0][1] & win32security.INHERITED_ACE) == win32security.INHERITED_ACE:
if not sidRet['sid'] or ace[2] == sidRet['sid']:
ret['Inheritance'] = True
return ret
break
ret['result'] = True
return ret