commit | 3de1bbfde9623ca4bf6d3a281d93857ae06e39a2 | [log] [tgz] |
---|---|---|
author | Ytai Ben-Tsvi <ytai@google.com> | Tue Jan 21 16:41:17 2020 -0800 |
committer | Ytai Ben-Tsvi <ytai@google.com> | Tue Jan 21 17:43:14 2020 -0800 |
tree | 6ef063e1db869e5a7e54cf6bebe7392ac64ae3e8 | |
parent | 31f98910412ed7e023ffb240c49de63f958fd226 [diff] |
Fix race condition in Effects Main problem was in AudioFlinger::EffectChain::EffectChain(), where the thread* argument could possibly have been destroyed before the creation of the weak pointer (AudioFlinger::EffectChain::EffectCallback::mThread) around it. While here, cleaned up a bunch of other pointer-related operations in order to avoid trafficking in raw pointers to RefBase subclasses. Bug: 147770363 Change-Id: I508ca94dd38a04e13f1a1c413f548001b961c721