Merge "Dont crash when test devices send bad messages. tested this on master with go/ag/4552612" (5ee35baa) · Commits · e / os / android_packages_modules_Bluetooth

android/app/src/com/android/bluetooth/mapclient/MceStateMachine.java

+9 −2

Original line number	Diff line number	Diff line
		@@ -456,8 +456,12 @@ final class MceStateMachine extends StateMachine {
		Log.d(TAG, "Message Sent......." + messageHandle);
		}
		// ignore the top-order byte (converted to string) in the handle for now
		// some test devices don't populate messageHandle field.
		// in such cases, no need to wait up for response for such messages.
		if (messageHandle != null && messageHandle.length() > 2) {
		mSentMessageLog.put(messageHandle.substring(2),
		((RequestPushMessage) message.obj).getBMsg());
		}
		} else if (message.obj instanceof RequestGetMessagesListing) {
		processMessageListing((RequestGetMessagesListing) message.obj);
		}
		@@ -605,6 +609,9 @@ final class MceStateMachine extends StateMachine {
		if (DBG) {
		Log.d(TAG, "got a status for " + handle + " Status = " + status);
		}
		// some test devices don't populate messageHandle field.
		// in such cases, ignore such messages.
		if (handle == null \|\| handle.length() <= 2) return;
		PendingIntent intentToSend = null;
		// ignore the top-order byte (converted to string) in the handle for now
		String shortHandle = handle.substring(2);