commit e912bfd79880ea5faac0fbcdb4e696125e151e18
author Jason Simmons <jsimmons@google.com> Wed Oct 31 10:50:12 2012 -0700
committer Eric Laurent <elaurent@google.com> Thu Nov 08 08:48:34 2012 -0800
tree b9803a98d67bd4c9160ff3ece52225bdc84d9623
parent 745e7fd4e09db2b47371338ee54c54255e286473

audio policy: direct output fix - DO NOT MERGE

merge "Close a newly opened direct output if its parameters are invalid"
from master

Change-Id: I27ac015217480a854d25273ab9498949971b411a

audio/AudioPolicyManagerBase.cpp

diff --git a/audio/AudioPolicyManagerBase.cpp b/audio/AudioPolicyManagerBase.cpp
index b4f4778..5d87399 100644
--- a/audio/AudioPolicyManagerBase.cpp
+++ b/audio/AudioPolicyManagerBase.cpp
@@ -1672,6 +1672,7 @@
                          ((profile->mFormats[0] == 0) &&
                              (profile->mChannelMasks.size() < 2))) {
                         ALOGW("checkOutputsForDevice() direct output missing param");
+                        mpClientInterface->closeOutput(output);
                         output = 0;
                     } else {
                         addOutput(output, desc);