commit 975ecc0e35526ed1c9de13155a99a98931810297
author Eric Laurent <elaurent@google.com> Fri May 19 16:27:45 2023 +0200
committer Eric Laurent <elaurent@google.com> Mon May 22 18:30:29 2023 +0000
tree f753c902a446794f22719f9da8dd4aaed342a724
parent ef8816464fb383427324369d87bfe7d2b0b2a69c

UsecaseValidator: ignore special use cases when usage is unknown

The game usecase validator must ignore special routing use cases
indicated by audio attributes flags (BT SCO, TTS...) when the usage
is unknown.

Also fix a problem in ToneGenerator where the attributes flags were
overwritten when the low latency flag was set.

Bug: 283014940
Test: repro steps in bug
Change-Id: I4d5a038436b0dd2453dd8320dafba478970bb06e

media/libaudiousecasevalidation/UsecaseValidator.cpp

diff --git a/media/libaudiousecasevalidation/UsecaseValidator.cpp b/media/libaudiousecasevalidation/UsecaseValidator.cpp
index d62df3a..bf532de 100644
--- a/media/libaudiousecasevalidation/UsecaseValidator.cpp
+++ b/media/libaudiousecasevalidation/UsecaseValidator.cpp
@@ -142,6 +142,9 @@
 
     bool areFlagsValid(audio_flags_mask_t flags) {
         ALOGV("areFlagsValid flags: %#x", flags);
+        if ((flags & (AUDIO_FLAG_SCO|AUDIO_FLAG_AUDIBILITY_ENFORCED|AUDIO_FLAG_BEACON)) != 0) {
+            return false;
+        }
         if ((flags & AUDIO_FLAG_LOW_LATENCY) != 0) {
             return true;
         }