commit 03611473bc1d13f8e55ac92e37a9716c2fd8e412
author Yuncheol Heo <ycheo@google.com> Thu Nov 06 17:21:04 2014 +0900
committer Yuncheol Heo <ycheo@google.com> Thu Nov 06 08:27:58 2014 +0000
tree 20759e55b987b7f261a8a655456d0464f63c2e7d
parent db66e1d2b9b63eb1612820dc6572de1e42880646

CEC: Allow no parameters with <System Audio Mode Request>.

Bug: 18266992
Change-Id: I4399e34d1178f3d4b17b4749ca543c2bb14c0765

services/core/java/com/android/server/hdmi/HdmiCecMessageValidator.java

diff --git a/services/core/java/com/android/server/hdmi/HdmiCecMessageValidator.java b/services/core/java/com/android/server/hdmi/HdmiCecMessageValidator.java
index f1529a7..53740fe 100644
--- a/services/core/java/com/android/server/hdmi/HdmiCecMessageValidator.java
+++ b/services/core/java/com/android/server/hdmi/HdmiCecMessageValidator.java
@@ -79,7 +79,7 @@
         addValidationInfo(Constants.MESSAGE_SET_STREAM_PATH,
                 physicalAddressValidator, DEST_BROADCAST);
         addValidationInfo(Constants.MESSAGE_SYSTEM_AUDIO_MODE_REQUEST,
-                physicalAddressValidator, DEST_DIRECT);
+                new SystemAudioModeRequestValidator(), DEST_DIRECT);
 
         // Messages have no parameter.
         FixedLengthValidator noneValidator = new FixedLengthValidator(0);
@@ -303,6 +303,17 @@
         }
     }
 
+    private class SystemAudioModeRequestValidator extends PhysicalAddressValidator {
+        @Override
+        public int isValid(byte[] params) {
+            // TV can send <System Audio Mode Request> with no parameters to terminate system audio.
+            if (params.length == 0) {
+                return OK;
+            }
+            return super.isValid(params);
+        }
+    }
+
     private class ReportPhysicalAddressValidator implements ParameterValidator {
         @Override
         public int isValid(byte[] params) {