commit 16d0a185142cff938d94f668e97cb95b4e742199
author Yabin Cui <yabinc@google.com> Mon Feb 22 22:53:52 2016 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Mon Feb 22 22:53:52 2016 +0000
tree bdf9bc64c890ad9af81aa5019b89cca00f2c60aa
parent fa9af7dae68aad09ed624a9786f6cd57713f7b7f
parent 53e7a0628f4acc95481f556ba51800df4a1de37d

Merge "recovery: check battery level before installing package."

minui/resources.cpp

diff --git a/minui/resources.cpp b/minui/resources.cpp
index fdbe554..8489d60 100644
--- a/minui/resources.cpp
+++ b/minui/resources.cpp
@@ -283,7 +283,7 @@
         goto exit;
     }
 
-    surface = reinterpret_cast<GRSurface**>(malloc(*frames * sizeof(GRSurface*)));
+    surface = reinterpret_cast<GRSurface**>(calloc(*frames, sizeof(GRSurface*)));
     if (surface == NULL) {
         result = -8;
         goto exit;
@@ -318,7 +318,7 @@
     if (result < 0) {
         if (surface) {
             for (int i = 0; i < *frames; ++i) {
-                if (surface[i]) free(surface[i]);
+                free(surface[i]);
             }
             free(surface);
         }
@@ -421,7 +421,7 @@
         png_read_row(png_ptr, row.data(), NULL);
         int w = (row[1] << 8) | row[0];
         int h = (row[3] << 8) | row[2];
-        int len = row[4];
+        __unused int len = row[4];
         char* loc = reinterpret_cast<char*>(&row[5]);
 
         if (y+1+h >= height || matches_locale(loc, locale)) {