diff options
| author | 2023-12-20 09:19:25 -0500 | |
|---|---|---|
| committer | 2023-12-20 09:58:17 -0500 | |
| commit | 66fdd6f352f074b55ef2eb81d22a1f41878453cb (patch) | |
| tree | 59ef8c5d1be67060bc823eba14ae39bfbec22f96 /libs/androidfw/FileStream.cpp | |
| parent | 77c5fef85b271c80da36dc24e92291934f01c3b8 (diff) | |
Use an Optional wrapper in TileLifecycleManager
This way, we can prevent NPE, as the variable is never null.
Aditionally, make sure that we always perform operations directly on
Optional.
It's hard to test the NPE because of thread issues, but with this fix we
can guarantee:
* No NPE thanks to Optional
* Behavior is maintained thanks to CTS
Test: atest TileLifecycleManagerTest
Test: atest CtsTileServiceTestCases
Test: atest CtsSystemUiHostTestCases
Fixes: 317014639
Flag: NONE
Change-Id: I27a5e5b4ced1fe5da9b0e4b686cb5e58ca820c8c
Diffstat (limited to 'libs/androidfw/FileStream.cpp')
0 files changed, 0 insertions, 0 deletions