commit def7ef9d3d2594864768a538f62b03e326e899a5
author Shikha Panwar <shikhapanwar@google.com> Fri Jan 06 08:35:48 2023 +0000
committer Shikha Panwar <shikhapanwar@google.com> Tue Jan 31 18:04:22 2023 +0000
tree fd452584cb7df84eef3972d9e47c0b2afd78603f
parent 39ee8bf670af115a7fa4d980b14775cdda6f5c1a

Sync encryptedstorage after payload execution

Microdroid manager will syncfs the encryptedstore's filesystem post
payload is finished.

This patch also include refactoring of TestService to include quit()
method so that the service can be gracefully shutdown triggering the
sync above^.

Test: atest MicrodroidTests#encryptedStorageIsPersistent
Bug: 264520098
Change-Id: I1c196fda1fb0818c30cc55a64603bebd1077d09b

tests/testapk/src/native/testbinary.cpp

diff --git a/tests/testapk/src/native/testbinary.cpp b/tests/testapk/src/native/testbinary.cpp
index 4ba502a..365ea75 100644
--- a/tests/testapk/src/native/testbinary.cpp
+++ b/tests/testapk/src/native/testbinary.cpp
@@ -240,9 +240,6 @@
                 return ScopedAStatus::fromExceptionCodeWithMessage(EX_SERVICE_SPECIFIC,
                                                                    msg.c_str());
             }
-            // TODO(b/264520098): Remove sync() once TestService supports quit() method
-            // and Microdroid manager flushes filesystem caches on shutdown.
-            sync();
             return ScopedAStatus::ok();
         }
 
@@ -255,6 +252,8 @@
             }
             return ScopedAStatus::ok();
         }
+
+        ScopedAStatus quit() override { exit(0); }
     };
     auto testService = ndk::SharedRefBase::make<TestService>();