commit 12bed86f41d7a4e53a442ac4ca181d1e39893059
author qctecmdr <qctecmdr@localhost> Wed Jun 19 08:49:29 2019 -0700
committer Gerrit - the friendly Code Review server <code-review@localhost> Wed Jun 19 08:49:29 2019 -0700
tree 2b3212fd5da499deed2b712322d6d0c0dc875053
parent 204a77c04813414b98f13d3e27e9913c30950b57
parent 2c9add7250b8fa0590f6106fc837510549f76367

Merge "audiopolicy: Check open outputs count before closing."

policy_hal/AudioPolicyManager.cpp

diff --git a/policy_hal/AudioPolicyManager.cpp b/policy_hal/AudioPolicyManager.cpp
index cb7db23..830ebe1 100644
--- a/policy_hal/AudioPolicyManager.cpp
+++ b/policy_hal/AudioPolicyManager.cpp
@@ -1795,7 +1795,8 @@
                     "format %d %d, channel mask %04x %04x", output, config->sample_rate,
                     outputDesc->mSamplingRate, config->format, outputDesc->mFormat,
                     channelMask, outputDesc->mChannelMask);
-            if (output != AUDIO_IO_HANDLE_NONE) {
+            //Only close o/p descriptor if successfully opened
+            if (status == NO_ERROR) {
                 outputDesc->close();
             }
             // fall back to mixer output if possible when the direct output could not be open