commit f876348c67c4efcab7241579ff161fc001156563
author Eric Laurent <elaurent@google.com> Wed Mar 30 20:37:30 2022 +0200
committer Eric Laurent <elaurent@google.com> Wed Mar 30 20:37:30 2022 +0200
tree 40abe26cb3babd86a62b327da1e9e6d85718e0bb
parent b7c00502628d3ff7c1de64460dfe92b7613a2559

audio policy: fix error log for permission checks

Fix a copy past error causing misleading error logs in case of
permission check failures.

Bug: 210136642
Test: make
Change-Id: I6a36d74c06b5a8a14a185161bdd1f4f4bcd122ce

services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp

diff --git a/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp b/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp
index ae4d174..c9cfbca 100644
--- a/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp
+++ b/services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp
@@ -90,13 +90,13 @@
             if (attr.usage == AUDIO_USAGE_CALL_ASSISTANT
                     && ((attr.flags & AUDIO_FLAG_CALL_REDIRECTION) != 0)) {
                 if (!callAudioInterceptionAllowed(attributionSource)) {
-                    ALOGE(("permission denied: modify audio routing not allowed "
-                           "for attributionSource %s"), attributionSource.toString().c_str());
+                    ALOGE("%s: call audio interception not allowed for attribution source: %s",
+                           __func__, attributionSource.toString().c_str());
                     return PERMISSION_DENIED;
                 }
             } else if (!modifyAudioRoutingAllowed(attributionSource)) {
-                ALOGE(("permission denied: modify audio routing not allowed "
-                       "for attributionSource %s"), attributionSource.toString().c_str());
+                ALOGE("%s: modify audio routing not allowed for attribution source: %s",
+                        __func__, attributionSource.toString().c_str());
                 return PERMISSION_DENIED;
             }
         } else {