fix: PR comment fixes

JimSycurity · JimSycurity · commit f23b9e047816 · 2025-07-28T14:32:16.000-05:00
diff --git a/src/CommonLib/Processors/ACLProcessor.cs b/src/CommonLib/Processors/ACLProcessor.cs
@@ -173,7 +173,7 @@ public bool IsACLProtected(byte[] ntSecurityDescriptor) {
         public bool? IsAdminSDHolderProtected(byte[] ntSecurityDescriptor, string adminSdHolderHash = null, string objectName = "") {
             bool? isAdminSdHolderProtected = null;
 
-            if (ntSecurityDescriptor == null || string.IsNullOrEmpty(adminSdHolderHash)) {
+            if (ntSecurityDescriptor == null || ntSecurityDescriptor.Length == 0 || string.IsNullOrEmpty(adminSdHolderHash)) {
                 _log.LogDebug("Required input(s) missing for AdminSDHolder hash comparison for object: {Name}", objectName);
                 return isAdminSdHolderProtected;
             }
@@ -246,7 +246,7 @@ public string CalculateImplicitACLHash(byte[] ntSecurityDescriptor, string objec
                 return string.Empty;
             }
 
-            _log.LogDebug("Calculating hash of implicit ACEs for {Name}", objectName);
+            _log.LogInformation("Calculating hash of implicit ACEs for {Name}", objectName);
             var descriptor = _utils.MakeSecurityDescriptor();
 
             try
@@ -263,7 +263,7 @@ public string CalculateImplicitACLHash(byte[] ntSecurityDescriptor, string objec
 
             // Check if DACL is protected
             bool isDaclProtected = descriptor.AreAccessRulesProtected();
-            _log.LogDebug("DACL Protection status for {Name}: {IsProtected}", objectName, isDaclProtected);
+            _log.LogInformation("DACL Protection status for {Name}: {IsProtected}", objectName, isDaclProtected);
 
             // Get all ACEs, including Deny ACEs, but skip inherited ones
             var aceList = new List<ACEForHashing>();
