Merge pull request #15 from YakDriver/nomapping

Fix account lookup issue

.bumpversion.cfg

@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 0.2.0
+current_version = 0.2.1
 commit = False
 tag = False
 tag_name = {new_version}

CHANGELOG.md

@@ -1,5 +1,15 @@
 ## Changelog
 
+### 0.2.1
+
+**Commit Delta**: [Change from 0.2.0 release](https://github.com/YakDriver/oschmod/compare/0.2.0...0.2.1)
+
+**Released**: 2020.04.20
+
+**Summary**:
+
+*   Fix issue with account lookup failing with error (`No mapping between account names and security IDs was done.`).
+
 ### 0.2.0
 
 **Commit Delta**: [Change from 0.1.5 release](https://github.com/YakDriver/oschmod/compare/0.1.5...0.2.0)

oschmod/__init__.py

@@ -152,7 +152,7 @@
     "S_IXOTH"
 )
 
-__version__ = "0.2.0"
+__version__ = "0.2.1"
 
 
 def get_mode(path):
@@ -349,7 +349,7 @@ def _win_set_permissions(path, mode, object_type):
     system_ace = None
     for _ in range(0, dacl.GetAceCount()):
         ace = dacl.GetAce(0)
-        if win32security.LookupAccountSid(
+        if ace[2] and ace[2].IsValid() and win32security.LookupAccountSid(
                 None, ace[2]) == SECURITY_NT_AUTHORITY:
             system_ace = ace
         dacl.DeleteAce(0)

setup.cfg

@@ -3,7 +3,7 @@ name = oschmod
 description = Windows and Linux compatible chmod
 long_description = file: README.md, CHANGELOG.md
 long_description_content_type = text/markdown
-version = 0.2.0
+version = 0.2.1
 author = YakDriver
 author_email = [email protected]
 url = https://github.com/yakdriver/oschmod