commit c6132d684e2e196d116e48b48f3a1b4cb66c2479
author jiabin <jiabin@google.com> Sat Jan 01 07:36:31 2022 +0000
committer Jiabin Huang <jiabin@google.com> Mon Jan 03 18:12:17 2022 +0000
tree 7012b10ffb502fc6c6aab1f83a5b92326231dc52
parent 187097e8b2440c345209841d63b61b5ef449468a

Fix direct mode query failure.

The root cause of the failure is a typo in the condition that makes
us wrongly return that offload is not supported when it is.

Test: atest AudioManagerTest AudioTrackOffloadTest
Bug: 212718725
Change-Id: I3eb9d6989481c5f835819180477a034d5d3788e6

services/audiopolicy/managerdefault/AudioPolicyManager.cpp

diff --git a/services/audiopolicy/managerdefault/AudioPolicyManager.cpp b/services/audiopolicy/managerdefault/AudioPolicyManager.cpp
index 2d36684..cc1012c 100644
--- a/services/audiopolicy/managerdefault/AudioPolicyManager.cpp
+++ b/services/audiopolicy/managerdefault/AudioPolicyManager.cpp
@@ -3797,7 +3797,7 @@
             }
             if ((curProfile->getFlags() & AUDIO_OUTPUT_FLAG_COMPRESS_OFFLOAD)
                         != AUDIO_OUTPUT_FLAG_NONE) {
-                if ((directMode | AUDIO_DIRECT_OFFLOAD_GAPLESS_SUPPORTED)
+                if ((directMode & AUDIO_DIRECT_OFFLOAD_GAPLESS_SUPPORTED)
                         != AUDIO_DIRECT_NOT_SUPPORTED) {
                     // Already reports offload gapless supported. No need to report offload support.
                     continue;