diff options
| author | Nikita Ioffe <ioffe@google.com> | 2020-11-27 18:57:44 +0000 |
|---|---|---|
| committer | Nikita Ioffe <ioffe@google.com> | 2020-12-01 22:54:05 +0000 |
| commit | 8b2501045c5a07ba3cffcaf45dbcccee7f651713 (patch) | |
| tree | 50fe1adc44506f224c35a738519997b436f72e04 /libutils/Unicode.cpp | |
| parent | 5931a428ec6fc5eca57d9633183d37537f2e7da0 (diff) | |
Fix potential use-after-free bug in reboot
Instead of operating on raw pointers, init now uses name of the
services as it's primary identifier. Only place that still uses
vector<Service*> is StopServices.
In addition, ServiceList::services() function is removed, which should
help avoiding similar bugs in the future.
Bug: 170315126
Bug: 174335499
Test: adb reboot
Test: atest CtsInitTestCases
Change-Id: I73ecd7a8c58c2ec3732934c595b7f7db814b7034
Merged-In: I73ecd7a8c58c2ec3732934c595b7f7db814b7034
(cherry picked from commit bbba64826fdfb7be1f04b7bbb125fbe6d25e5028)
Diffstat (limited to 'libutils/Unicode.cpp')
0 files changed, 0 insertions, 0 deletions