Fix redundant codeflow in openLogicalChannel with UICC terminal

When setup channel access for UICC terminal, carrier privileges checking
add unnecessary rules loading.

1 files changed, 17 insertions, 17 deletions

diff --git a/src/com/android/se/Terminal.java b/src/com/android/se/Terminal.java
index 1dc3ed7..8147601 100755
--- a/src/com/android/se/Terminal.java
+++ b/src/com/android/se/Terminal.java
@@ -805,9 +805,23 @@ public class Terminal {
         }
         mAccessControlEnforcer.setPackageManager(mContext.getPackageManager());
 
-        if (getName().startsWith(SecureElementService.UICC_TERMINAL)
-                && isCarrierPrivilegeApplication(packageName)) {
-            return ChannelAccess.getCarrierPrivilegeAccess(packageName, pid);
+        if (getName().startsWith(SecureElementService.UICC_TERMINAL)) {
+            try {
+                PackageManager pm = mContext.getPackageManager();
+                if (pm != null) {
+                    PackageInfo pkgInfo =
+                            pm.getPackageInfo(packageName, PackageManager.GET_SIGNATURES);
+                    if (mAccessControlEnforcer.checkCarrierPrivilege(pkgInfo, checkRefreshTag)) {
+                        Log.i(mTag, "setUp PrivilegeAccess for CarrierPrivilegeApplication. ");
+                        return ChannelAccess.getCarrierPrivilegeAccess(packageName, pid);
+                    }
+                    checkRefreshTag = false;
+                }
+            } catch (NameNotFoundException ne) {
+                Log.e(mTag, "checkCarrierPrivilege(): packageInfo is not found. ");
+            } catch (Exception e) {
+                Log.e(mTag, "checkCarrierPrivilege() Exception: " + e.getMessage());
+            }
         }
 
         synchronized (mLock) {
@@ -870,20 +884,6 @@ public class Terminal {
     /**
      * Checks if Carrier Privilege exists for the given package
      */
-    private boolean isCarrierPrivilegeApplication(String packageName) {
-        try {
-            PackageManager pm = mContext.getPackageManager();
-            if (pm != null) {
-                PackageInfo pkgInfo = pm.getPackageInfo(packageName, PackageManager.GET_SIGNATURES);
-                return checkCarrierPrivilegeRules(pkgInfo);
-            }
-        } catch (NameNotFoundException ne) { }
-        return false;
-    }
-
-    /**
-     * Checks if Carrier Privilege exists for the given package
-     */
     public boolean checkCarrierPrivilegeRules(PackageInfo pInfo) {
         boolean checkRefreshTag = true;
         if (mAccessControlEnforcer == null || mAccessControlEnforcer.isNoRuleFound()) {