diff options
| author | Jean-Michel Trivi <jmtrivi@google.com> | 2018-06-07 12:28:27 -0700 |
|---|---|---|
| committer | Jean-Michel Trivi <jmtrivi@google.com> | 2018-06-07 12:28:27 -0700 |
| commit | c106d9f958be76ab14a6724525f39bc3fec03fd2 (patch) | |
| tree | fd62c6b49b07039fd6d0e085c297ed9f0fccc17d /tools/aapt2/java/JavaClassGenerator_test.cpp | |
| parent | c6e4acc222ca99af82e3910d164cdf3f5636559b (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
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator_test.cpp')
0 files changed, 0 insertions, 0 deletions