From 49553c75612acf2e0c8731d9d9eff26dc0f18a19 Mon Sep 17 00:00:00 2001
From: Koushik Dutta <koushd@gmail.com>
Date: Fri, 13 May 2011 13:01:26 -0700
Subject: [PATCH] Do not close the zip archive prematurely. It may be accessed
 later for a firmware update.

Change-Id: I31c298f75bbcdc7998221aa2b3aa334926343139
---
 install.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/install.c b/install.c
index ad2c21d..8194c35 100644
--- a/install.c
+++ b/install.c
@@ -127,10 +127,10 @@ try_update_binary(const char *path, ZipArchive *zip) {
     }
     bool ok = mzExtractZipEntryToFile(zip, binary_entry, fd);
     close(fd);
-    mzCloseZipArchive(zip);
 
     if (!ok) {
         LOGE("Can't copy %s\n", ASSUMED_UPDATE_BINARY_NAME);
+        mzCloseZipArchive(zip);
         return 1;
     }
 
@@ -239,13 +239,13 @@ try_update_binary(const char *path, ZipArchive *zip) {
     waitpid(pid, &status, 0);
     if (!WIFEXITED(status) || WEXITSTATUS(status) != 0) {
         LOGE("Error in %s\n(Status %d)\n", path, WEXITSTATUS(status));
+        mzCloseZipArchive(zip);
         return INSTALL_ERROR;
     }
 
     if (firmware_type != NULL) {
+        mzCloseZipArchive(zip);
         return handle_firmware_update(firmware_type, firmware_filename, zip);
-    } else {
-        return INSTALL_SUCCESS;
     }
     return INSTALL_SUCCESS;
 }