commit | 9bc7d36c505d7315f71c82437b2f082c376eb0ad | [log] [tgz] |
---|---|---|
author | Rambo Wang <rambowang@google.com> | Tue Mar 09 09:10:58 2021 -0800 |
committer | Rambo Wang <rambowang@google.com> | Tue Mar 09 10:49:29 2021 -0800 |
tree | 1232a0d60bb109b81361eaa8f668ef335f0e2661 | |
parent | e310dc7921016f5144b9652af4756f7f5c8b4692 [diff] |
Shared state modified in two threads without synchronization mConfigFromCarrierApp array is the shared state of CarrierConfigLoader. It is modified in both IPC thread and the handler thread without synchronization. This may cause data race problem. Move the modification of mConfigFromCarrierApp into the handler thread on which all shared states are designed to confine. Bug: 181979319 Test: make Change-Id: I6777af2c4798f6c98403b7e5ec3af47568cce916