diff options
author | Jack Yu <jackcwyu@google.com> | 2020-04-15 16:55:04 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2020-04-15 16:55:04 +0000 |
commit | cbddd0c10e9bcedbb713cddeb421b5792fd8a5e0 (patch) | |
tree | ef22dfd86674c7c12eb81922c072c7f7071824e5 | |
parent | d746b735df250b9ae587df938a0337f6bc39fccf (diff) | |
parent | 054af27cabc214a987804bc4be0e970896e8a002 (diff) | |
download | SecureElement-cbddd0c10e9bcedbb713cddeb421b5792fd8a5e0.tar.gz |
Fix redundant codeflow in openLogicalChannel with UICC terminal am: 054af27cab
Change-Id: Iae86ee1b1d99725384774bc0f31b5ba04e34f483
-rw-r--r-- | src/com/android/se/Terminal.java | 34 |
1 files changed, 17 insertions, 17 deletions
diff --git a/src/com/android/se/Terminal.java b/src/com/android/se/Terminal.java index 02ffd1a..69b8368 100644 --- a/src/com/android/se/Terminal.java +++ b/src/com/android/se/Terminal.java @@ -717,9 +717,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) { @@ -782,20 +796,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()) { |