diff options
| author | Jean-Michel Trivi <jmtrivi@google.com> | 2018-06-07 12:28:27 -0700 |
|---|---|---|
| committer | Eric Arseneau <earseneau@google.com> | 2018-06-12 10:26:13 -0700 |
| commit | 1729c580310448f1cf04028619e3273511309a49 (patch) | |
| tree | b3970f43c6d7063792166e26408095b43d319725 /tools/aapt2/java/JavaClassGenerator.cpp | |
| parent | 75bced48f9d622c32634f256c238e3f30580abed (diff) | |
AudioService: fix deallock setBluetoothScoOn vs setMode
In AudioService:
setBluetoothScoOn()
> setBluetoothScoOnInt()
> synchronized(mSettingsLock)
> muteRingerModeStreams()
> synchronized (VolumeStreamState.class)
vs
setMode()
> setModeInt()
> synchronized (mSettingsLock)
> synchronized (VolumeStreamState.class)
The issue comes from setBluetoothScoOnInt() releasing the lock
on mSettingsLock before calling muteRingerModeStreams(). This
breaks the expected lock ordering in AudioService, where
mSettingsLock must be locked before VolumeStreamState.class.
Bug: 109746821
Test: see bug
Change-Id: I853ca60781efc506626b6345f27da701655ac359
(cherry picked from commit c106d9f958be76ab14a6724525f39bc3fec03fd2)
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator.cpp')
0 files changed, 0 insertions, 0 deletions