| commit | 5eb07e3b88f64e7eb2a6e03b7e3265f647d80cb5 | [log] [tgz] |
|---|---|---|
| author | Jason Macnak <natsu@google.com> | Tue Oct 01 15:40:33 2024 -0700 |
| committer | Nolan Scobie <nscobie@google.com> | Wed Oct 16 18:18:21 2024 +0000 |
| tree | 1da40faea427203db7ad60e32e5b2b141ed97d76 | |
| parent | f5a213c5ed5056cf3f8a3b1aba4ade38294f947d [diff] [blame] |
Avoid leaking RenderEngine ... as this might allow an underlying `RenderEngineThreaded` thread to race. Bug: b/368983946 Test: stress test of VtsHalGraphicsComposer3_TargetTest Bug: b/361574971 Test: ATP cuttlefish_gfxstream_guest_angle_graphics_vts_tests (cherry picked from https://android-review.googlesource.com/q/commit:64f73a4c5fdc79a882c9047e69d9f64b89c013b9) Merged-In: Ief71d8931ec3560780a850d8bb8bc727bbff5c52 Change-Id: Ief71d8931ec3560780a850d8bb8bc727bbff5c52
diff --git a/graphics/composer/aidl/vts/RenderEngineVts.cpp b/graphics/composer/aidl/vts/RenderEngineVts.cpp index 66779c8..d0bc689 100644 --- a/graphics/composer/aidl/vts/RenderEngineVts.cpp +++ b/graphics/composer/aidl/vts/RenderEngineVts.cpp
@@ -29,9 +29,7 @@ mRenderEngine = ::android::renderengine::RenderEngine::create(args); } -TestRenderEngine::~TestRenderEngine() { - mRenderEngine.release(); -} +TestRenderEngine::~TestRenderEngine() {} void TestRenderEngine::setRenderLayers(std::vector<std::shared_ptr<TestLayer>> layers) { sort(layers.begin(), layers.end(),