commit cdcfa29b21ad921035e4cf0c99c40b9fca032006
author Hall Liu <hallliu@google.com> Thu May 02 09:16:29 2019 -0700
committer Hall Liu <hallliu@google.com> Fri May 03 10:39:48 2019 -0700
tree 76c0c807d07cbede11e4a7442e6d435107f6963f
parent 560554d976a3ee19c85abbbc5a1761ee966019f4

Don't allow channel 0 for transmit apdu channel

When iccTransmitApduLogicalChannel gets called with channel=0, return
an empty string instead of proceeding, since this crashes the phone
process.

Bug: 131401531
Test: CTS
Change-Id: I92eb087d3c6e57687f3066150c88cb948c4fc746

src/com/android/phone/PhoneInterfaceManager.java

diff --git a/src/com/android/phone/PhoneInterfaceManager.java b/src/com/android/phone/PhoneInterfaceManager.java
index 46c2847..22402d3 100755
--- a/src/com/android/phone/PhoneInterfaceManager.java
+++ b/src/com/android/phone/PhoneInterfaceManager.java
@@ -3813,7 +3813,7 @@
             int command, int p1, int p2, int p3, String data) {
         final long identity = Binder.clearCallingIdentity();
         try {
-            if (channel < 0) {
+            if (channel <= 0) {
                 return "";
             }