commit 90f5d648489ca8304d7c3ca8995e4e0b9ddab6cb
author Mikhail Lappo <miklelappo@gmail.com> Thu Mar 23 22:32:30 2017 +0100
committer Mikhail Lappo <miklelappo@gmail.com> Thu Mar 23 22:41:42 2017 +0100
tree be6f17d5c3bff674d92f441fd80675cc1b0fcb1e
parent 26464914feccfce4b835bd73a1130ba5be069e8a

Va_end should be used with va_start

va_start is used and va_end should
be invoked accordingly

Change-Id: Id7037aa8092438b27e445e5147972836602bfd55

libutils/Printer.cpp

diff --git a/libutils/Printer.cpp b/libutils/Printer.cpp
index 98cd2c6..84af293 100644
--- a/libutils/Printer.cpp
+++ b/libutils/Printer.cpp
@@ -47,9 +47,11 @@
 #ifndef _WIN32
     if (vasprintf(&formattedString, format, arglist) < 0) { // returns -1 on error
         ALOGE("%s: Failed to format string", __FUNCTION__);
+        va_end(arglist);
         return;
     }
 #else
+    va_end(arglist);
     return;
 #endif