fix: break after determining file access level

codebytere · Feb 9, 2023 · 04bc433 · 04bc433
1 parent 1c28b7b
commit 04bc433
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/permissions.mm b/permissions.mm
@@ -279,8 +279,8 @@ bool HasOpenSystemPreferencesDialog() {
   for (NSString *file in files) {
     const std::string can_read = CheckFileAccessLevel(file);
     if (can_read == kAuthorized) {
-      break;
       auth_status = kAuthorized;
+      break;
     } else if (can_read == kDenied) {
       auth_status = kDenied;
     }
@@ -294,11 +294,7 @@ bool HasOpenSystemPreferencesDialog() {
 std::string ScreenAuthStatus() {
   std::string auth_status = kNotDetermined;
   if (@available(macOS 10.16, *)) {
-    if (CGPreflightScreenCaptureAccess()) {
-      auth_status = kAuthorized;
-    } else {
-      auth_status = kDenied;
-    }
+    auth_status = CGPreflightScreenCaptureAccess() ? kAuthorized : kDenied;
   } else if (@available(macOS 10.15, *)) {
     auth_status = kDenied;
     NSRunningApplication *runningApplication =