Change optional parameters to required

SpecterOps · Dec 5, 2024 · e92a533 · e92a533
1 parent 6859953
commit e92a533
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/CommonLib/Processors/ACLProcessor.cs b/src/CommonLib/Processors/ACLProcessor.cs
@@ -235,7 +235,7 @@ public IAsyncEnumerable<ACE> ProcessACL(ResolvedSearchResult result, IDirectoryO
             {
                 return (Array.Empty<ACE>(), false, false);
             }
-            return await ProcessACL(descriptor, result.Domain, result.ObjectType, searchResult.HasLAPS(), result.DisplayName, checkForOwnerRights);
+            return await ProcessACL(descriptor, result.Domain, result.ObjectType, searchResult.HasLAPS(), checkForOwnerRights, result.DisplayName);
         }
 
         /// <summary>
@@ -251,15 +251,15 @@ public IAsyncEnumerable<ACE> ProcessACL(ResolvedSearchResult result, IDirectoryO
         public async IAsyncEnumerable<ACE> ProcessACL(byte[] ntSecurityDescriptor, string objectDomain,
            Label objectType, bool hasLaps, string objectName = "")
         {
-            var (aces, _, _) = await ProcessACL(ntSecurityDescriptor, objectDomain, objectType, hasLaps, objectName, true);
+            var (aces, _, _) = await ProcessACL(ntSecurityDescriptor, objectDomain, objectType, hasLaps, true, objectName);
             foreach (var ace in aces)
             {
                 yield return ace;
             }
         }
 
         public async Task<(ACE[], bool, bool)> ProcessACL(byte[] ntSecurityDescriptor, string objectDomain,
-            Label objectType, bool hasLaps, string objectName = "", bool checkForOwnerRights = true)
+            Label objectType, bool hasLaps, bool checkForOwnerRights, string objectName)
         {
             var aces = new List<ACE>();
             bool isAnyPermissionForOwnerRightsSid = false;