commit 0af756cd6f67fe1b615a3a65451069a730bdb0d2
author Treehugger Robot <treehugger-gerrit@google.com> Thu May 26 02:38:08 2022 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Thu May 26 02:38:08 2022 +0000
tree 2a39005fa7ad5eb418101bbc935a699cafea182a
parent 67c4aa8d0abf3377b20be9348f6cc2b70ddaa0b9
parent 1ec805514b1a0759e54d2b70433ddf8a427d0eb6

Merge "Fix memory leak in OTA verifier code" am: 65162cd9b0 am: efc4279944 am: 6e0b7fa5ec am: 1ec805514b

Original change: https://android-review.googlesource.com/c/platform/bootable/recovery/+/2106828

Change-Id: Ic6fff9e16eea28bcd72fa96b218d3b2dd5ea5dd8
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

otautil/verifier.cpp

diff --git a/otautil/verifier.cpp b/otautil/verifier.cpp
index 8a65566..83ad708 100644
--- a/otautil/verifier.cpp
+++ b/otautil/verifier.cpp
@@ -309,13 +309,15 @@
 }
 
 static std::vector<Certificate> IterateZipEntriesAndSearchForKeys(const ZipArchiveHandle& handle) {
-  void* cookie;
+  void* cookie{};
+
   int32_t iter_status = StartIteration(handle, &cookie, "", "x509.pem");
   if (iter_status != 0) {
     LOG(ERROR) << "Failed to iterate over entries in the certificate zipfile: "
                << ErrorCodeString(iter_status);
     return {};
   }
+  std::unique_ptr<void, decltype(&EndIteration)> cookie_guard(cookie, &EndIteration);
 
   std::vector<Certificate> result;