summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorandroid-build-team Robot <android-build-team-robot@google.com>2020-06-11 01:01:02 +0000
committerandroid-build-team Robot <android-build-team-robot@google.com>2020-06-11 01:01:02 +0000
commit255b5be5c7e45be103f855dd4732c227b7a0c24a (patch)
tree28e6403c0f813d72307dd23d45976438a1a81089
parentc0a6e883d2ea8841a0da93e2f7bc7e739214b64b (diff)
parentda6daab1ebf75fef31bf78d01f1f1f76903a3f73 (diff)
downloadSecureElement-255b5be5c7e45be103f855dd4732c227b7a0c24a.tar.gz
Snap for 6578268 from da6daab1ebf75fef31bf78d01f1f1f76903a3f73 to rvc-release
Change-Id: Ia313fd6608517863db2a58c8a9e069749199ae6d
-rw-r--r--src/com/android/se/SecureElementService.java3
-rw-r--r--src/com/android/se/Terminal.java6
2 files changed, 5 insertions, 4 deletions
diff --git a/src/com/android/se/SecureElementService.java b/src/com/android/se/SecureElementService.java
index 5ca4646..e979fd9 100644
--- a/src/com/android/se/SecureElementService.java
+++ b/src/com/android/se/SecureElementService.java
@@ -303,9 +303,6 @@ public final class SecureElementService extends Service {
throw new IllegalStateException("Session is closed");
} else if (listener == null) {
throw new NullPointerException("listener must not be null");
- } else if (((aid == null) || (aid.length == 0)) && mReader.getTerminal().getName()
- .startsWith(SecureElementService.UICC_TERMINAL)) {
- return null;
} else if ((p2 != 0x00) && (p2 != 0x04) && (p2 != 0x08)
&& (p2 != (byte) 0x0C)) {
throw new UnsupportedOperationException("p2 not supported: "
diff --git a/src/com/android/se/Terminal.java b/src/com/android/se/Terminal.java
index 080f337..4638926 100644
--- a/src/com/android/se/Terminal.java
+++ b/src/com/android/se/Terminal.java
@@ -517,7 +517,7 @@ public class Terminal {
packageName);
try {
channelAccess = setUpChannelAccess(aid, packageName, pid, false);
- } catch (MissingResourceException e) {
+ } catch (MissingResourceException | UnsupportedOperationException e) {
return null;
}
}
@@ -761,6 +761,10 @@ public class Terminal {
}
if (isBasicChannel) {
throw new MissingResourceException("openBasicChannel is not allowed.", "", "");
+ } else if (aid == null) {
+ // openLogicalChannel with null aid is only allowed for privilege applications
+ throw new UnsupportedOperationException(
+ "null aid is not accepted in UICC terminal.");
}
}