| commit | 1f65b3a3eba7a502b6a2318734cf293b27e8bc50 | [log] [tgz] |
|---|---|---|
| author | sergiuferentz <sergiuferentz@google.com> | Mon Jun 26 18:01:47 2023 +0000 |
| committer | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | Thu Aug 10 17:11:56 2023 +0000 |
| tree | d930a0b49587007cfdaacd461c7197c156393600 | |
| parent | 0b0348126669c73db19990d4315f9691df49f168 [diff] |
Fix for heap-use-after-free in GPUService.cpp This adds a unit test and fix for the bug reported by libfuzzer. Changes made: * Expose GPUService as testable code. * Update main_gpuservice.cpp to use the new GpuService now located at gpuservice/GpuService.h * Make initializer threads members of GpuService * Join the threads in destructor to prevent heap-use-after-free. * Add unit test that waits 3 seconds after deallocation to ensure no wrong access is made. Bug: 282919145 Test: Added unit test and ran on device with ASAN (cherry picked from commit 3c00cbc0f119c3f59325aa6d5061529feb58462b) (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:24a7874bb4093a1a6721a2002569512c43af5bdc) Merged-In: I4d1d2d4658b575bf2c8f425f91f68f03114ad029 Change-Id: I4d1d2d4658b575bf2c8f425f91f68f03114ad029