commit 17031564f347330f7b34dda59e82be78ba0e4abd
author Mikhail Naganov <mnaganov@google.com> Wed Feb 23 23:00:27 2022 +0000
committer Mikhail Naganov <mnaganov@google.com> Wed Feb 23 23:05:35 2022 +0000
tree 1424bc40a5d8ddecd0c944528fb440d7d4074106
parent 04f3e76b78702f620c435e6a72220ede72a21e33

Fix AudioSystem::getAudioPort implementation

The implementation used to try to convert the complete provided
audio port into AIDL before passing to AP Service. The
corresponding AP Service method has been changed to only take
the port ID—the only piece of information which is actually
used by APM.

Bug: 221075345
Test: in preparation by a contractor
Change-Id: Id274ee14a8bfd2dcbfbb22b7a09b89dc6b7314ce

services/audiopolicy/service/AudioPolicyService.h

diff --git a/services/audiopolicy/service/AudioPolicyService.h b/services/audiopolicy/service/AudioPolicyService.h
index 4944497..0a01b7b 100644
--- a/services/audiopolicy/service/AudioPolicyService.h
+++ b/services/audiopolicy/service/AudioPolicyService.h
@@ -176,7 +176,7 @@
     binder::Status listAudioPorts(media::AudioPortRole role, media::AudioPortType type,
                                   Int* count, std::vector<media::AudioPort>* ports,
                                   int32_t* _aidl_return) override;
-    binder::Status getAudioPort(const media::AudioPort& port,
+    binder::Status getAudioPort(int portId,
                                 media::AudioPort* _aidl_return) override;
     binder::Status createAudioPatch(const media::AudioPatch& patch, int32_t handle,
                                     int32_t* _aidl_return) override;