commit e31f06a0e1d0704cc25c588158c57cf94077a5fd
author kunal rai <kunal.rai@ittiam.com> Tue Oct 18 12:18:18 2022 +0530
committer Ayushi Khopkar <akhopkar@google.com> Tue Oct 18 08:54:07 2022 +0000
tree e1ff0a109367c4cd2f0dcfebf10282c496d2c74e
parent 9532a51ce1a8b500099da0af50fccc3fc0bef679

surfaceflinger_fuzzer : Resolved memory leak

Test: ./surfaceflinger_fuzzer
Bug: 251848241

Change-Id: I7a470f276b5886f86b80faa1827249c4265aa0fe
(cherry picked from commit cbcf790edab2306e4b55e441111592ff5e77c611)

services/surfaceflinger/fuzzer/surfaceflinger_fuzzer.cpp

diff --git a/services/surfaceflinger/fuzzer/surfaceflinger_fuzzer.cpp b/services/surfaceflinger/fuzzer/surfaceflinger_fuzzer.cpp
index 22d80ca..a3da4dc 100644
--- a/services/surfaceflinger/fuzzer/surfaceflinger_fuzzer.cpp
+++ b/services/surfaceflinger/fuzzer/surfaceflinger_fuzzer.cpp
@@ -245,6 +245,7 @@
     setDisplayStateLocked();
 
     setTransactionState();
+    mTestableFlinger.flushTransactionQueues();
 
     onTransact(data, size);
 }

services/surfaceflinger/fuzzer/surfaceflinger_fuzzers_utils.h

diff --git a/services/surfaceflinger/fuzzer/surfaceflinger_fuzzers_utils.h b/services/surfaceflinger/fuzzer/surfaceflinger_fuzzers_utils.h
index ed2fe23..ee48345 100644
--- a/services/surfaceflinger/fuzzer/surfaceflinger_fuzzers_utils.h
+++ b/services/surfaceflinger/fuzzer/surfaceflinger_fuzzers_utils.h
@@ -743,6 +743,11 @@
                                              listenerCallbacks, transactionId);
     }
 
+    auto flushTransactionQueues() {
+        ftl::FakeGuard guard(kMainThreadContext);
+        return mFlinger->flushTransactionQueues(VsyncId{0});
+    }
+
     auto onTransact(uint32_t code, const Parcel &data, Parcel *reply, uint32_t flags) {
         return mFlinger->onTransact(code, data, reply, flags);
     }