Merge pull request #85 from grahamgilbert/mbsetupuser

Check for _mbsetupuser before checking for the pref

Author: grahamgilbert

Crypt/Info.plist

@@ -15,11 +15,11 @@
 	<key>CFBundlePackageType</key>
 	<string>BNDL</string>
 	<key>CFBundleShortVersionString</key>
-	<string>3.2.0</string>
+	<string>3.2.1</string>
 	<key>CFBundleSignature</key>
 	<string>????</string>
 	<key>CFBundleVersion</key>
-	<string>3.2.0</string>
+	<string>3.2.1</string>
 	<key>NSHumanReadableCopyright</key>
 	<string>Copyright © 2018 The Crypt Project. All rights reserved.</string>
 	<key>NSPrincipalClass</key>

Crypt/Mechanisms/Check.swift

@@ -212,7 +212,21 @@ class Check: CryptMechanism {
   fileprivate func getSkipUsers() -> Bool {
     os_log("Checking for any SkipUsers...", log: Check.log, type: .default)
     guard let username = self.username
-      else { return false }
+      else { 
+        os_log("Cannot get username", log: Check.log, type: .error)
+        return false 
+      }
+    os_log("Username is %{public}@...", log: Check.log, type: .error, String(describing: username))
+
+    if username as String == "_mbsetupuser" {
+      os_log("User is _mbsetupuser... Need to Skip...", log: Check.log, type: .error)
+      return true
+    }
+
+    if username as String == "root" {
+      os_log("User is root... Need to Skip...", log: Check.log, type: .error)
+      return true
+    }
     guard let prefValue = CFPreferencesCopyAppValue("SkipUsers" as CFString, bundleid as CFString) as? [String]
       else { return false }
     for s in prefValue {
@@ -221,10 +235,6 @@ class Check: CryptMechanism {
         return true
       }
     }
-    if username as String == "_mbsetupuser" {
-      os_log("User is _mbsetupuser... Need to Skip...", log: Check.log, type: .error)
-      return true
-    }
     return false
   }