Merge "Prevent modifying mAudioPolicies during iteration" into tm-qpr-dev am: 7926ae7f37
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/19487769 Change-Id: I0d8d8e86c65d331fb2e19f3083c881732b87b7e7 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -1614,6 +1614,7 @@ public class AudioService extends IAudioService.Stub
|
|||||||
}
|
}
|
||||||
|
|
||||||
synchronized (mAudioPolicies) {
|
synchronized (mAudioPolicies) {
|
||||||
|
ArrayList<AudioPolicyProxy> invalidProxies = new ArrayList<>();
|
||||||
for (AudioPolicyProxy policy : mAudioPolicies.values()) {
|
for (AudioPolicyProxy policy : mAudioPolicies.values()) {
|
||||||
final int status = policy.connectMixes();
|
final int status = policy.connectMixes();
|
||||||
if (status != AudioSystem.SUCCESS) {
|
if (status != AudioSystem.SUCCESS) {
|
||||||
@@ -1621,7 +1622,7 @@ public class AudioService extends IAudioService.Stub
|
|||||||
Log.e(TAG, "onAudioServerDied: error "
|
Log.e(TAG, "onAudioServerDied: error "
|
||||||
+ AudioSystem.audioSystemErrorToString(status)
|
+ AudioSystem.audioSystemErrorToString(status)
|
||||||
+ " when connecting mixes for policy " + policy.toLogFriendlyString());
|
+ " when connecting mixes for policy " + policy.toLogFriendlyString());
|
||||||
policy.release();
|
invalidProxies.add(policy);
|
||||||
} else {
|
} else {
|
||||||
final int deviceAffinitiesStatus = policy.setupDeviceAffinities();
|
final int deviceAffinitiesStatus = policy.setupDeviceAffinities();
|
||||||
if (deviceAffinitiesStatus != AudioSystem.SUCCESS) {
|
if (deviceAffinitiesStatus != AudioSystem.SUCCESS) {
|
||||||
@@ -1629,10 +1630,12 @@ public class AudioService extends IAudioService.Stub
|
|||||||
+ AudioSystem.audioSystemErrorToString(deviceAffinitiesStatus)
|
+ AudioSystem.audioSystemErrorToString(deviceAffinitiesStatus)
|
||||||
+ " when connecting device affinities for policy "
|
+ " when connecting device affinities for policy "
|
||||||
+ policy.toLogFriendlyString());
|
+ policy.toLogFriendlyString());
|
||||||
policy.release();
|
invalidProxies.add(policy);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
invalidProxies.forEach((policy) -> policy.release());
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// Restore capture policies
|
// Restore capture policies
|
||||||
|
|||||||
Reference in New Issue
Block a user