diff options
| author | Christopher Wiley <wiley@google.com> | 2016-05-31 10:44:35 -0700 |
|---|---|---|
| committer | Mitchell Wills <mwills@google.com> | 2016-08-26 13:22:47 -0700 |
| commit | e90e0a74adf33f3d0ed0f2617520ab72b3b5d624 (patch) | |
| tree | 927b6fb2008c6bde554534c7c792e69915a9436c /opengl/tests/gl_basic/gl_basic.cpp | |
| parent | f9f88b8b20f4fe4e0c7b2c22c1f0d4d0a36f2217 (diff) | |
Fix race conditions between Tethering and TetherInterfaceStateMachine
Previously, Tethering would answer requests for current tethering state
by calling methods on instances of TetherInterfaceStateMachine to build
up that state. This is incorrect, since state queries can come in on an
arbitrary thread, independent of updates to the state machines
instances.
Fix this by:
- Altering TetherInterfaceStateMachine to push consistent state
snapshots to Tethering via notifyInterfaceStateChange()
- Storing the last state snapshot in Tethering.mTetherStates
- Removing public methods to query TetherInterfaceStateMachine state.
- Consistently synchronizing access to Tethering.mTetherStates
Bug: 29009601
Test: WiFi Tethering continues to work, unittests continue to pass
Change-Id: Ied334f5e8739bc3aff1b08a1079095b9cc2a7958
(cherry picked from commit de4819dc82b734036b22416228327d2712e01884)
Diffstat (limited to 'opengl/tests/gl_basic/gl_basic.cpp')
0 files changed, 0 insertions, 0 deletions