renaming "data" to "computer", fixes #3

c3c · Jan 3, 2022 · 977607f · 977607f
1 parent 33708fa
commit 977607f
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/adexpsnapshot/__init__.py b/adexpsnapshot/__init__.py
@@ -222,7 +222,6 @@ def processComputers(self, entry):
         distinguishedName = ADUtils.get_entry_property(entry, 'distinguishedName')
         domain = ADUtils.ldap2domain(distinguishedName)
 
-        samname = ADUtils.get_entry_property(entry, 'sAMAccountName')
         primarygroup = MembershipEnumerator.get_primary_membership(entry)
 
         computer = {
@@ -272,10 +271,10 @@ def processComputers(self, entry):
                 continue
             try:
                 sid = self.computersidcache.get(target)
-                data['AllowedToDelegate'].append(sid)
+                computer['AllowedToDelegate'].append(sid)
             except KeyError:
                 if '.' in target:
-                    data['AllowedToDelegate'].append(target.upper())
+                    computer['AllowedToDelegate'].append(target.upper())
         if len(delegatehosts) > 0:
             props['allowedtodelegate'] = delegatehosts