diff options
| author | Charles Chen <charlesccchen@google.com> | 2020-03-11 10:17:19 +0800 |
|---|---|---|
| committer | Charles Chen <charlesccchen@google.com> | 2020-04-08 21:30:24 +0800 |
| commit | 4bff5be1a91e7598731412981ed512c9060ef188 (patch) | |
| tree | 16f9ca5d4a184144776613c5a1c6c1b2e67ffe90 /rs/java/android/renderscript/ProgramFragmentFixedFunction.java | |
| parent | 884aa6afe156ec88cfc89e9d5dac4480223c9495 (diff) | |
Fix WindowContext leak
The root causes of this issue are:
1. WindowTokenClient#attachContext makes WindowTokenClient has strong
reference to WindowContext, which leads to WindowContext cannot be
GC'd.
2. WMS#removeWindowToken needs MANAGE_APP_TOKEN permission which
normal apps don't hold.
This CL does following things:
1. Use weak reference instead on WindowTokenClient#mContext.
2. Relax WMS#removeWindowToken to check callingUid if
MANAGE_WINDOW_TOKEN permission is not held
3. Deliver config changes to the client side in
WMS#addWindowTokenWithOption
4. Some minor fixes
fixes: 150812449
Bug: 150715095
Test: atest WindowContextTest
Test: atest WindowManagerServiceTests
Test: atest WindowTokenTests
Test: atest WindowManagerPermissionTests#testMANAGE_APP_TOKENS
Test: atest
WindowManagerPermissionTests#testADD_WINDOW_TOKEN_WITH_OPTIONS
Change-Id: I9f1d73af2abb78fc9844e6d9eb25e9f0293514e7
Diffstat (limited to 'rs/java/android/renderscript/ProgramFragmentFixedFunction.java')
0 files changed, 0 insertions, 0 deletions