commit 92157cddd13ca1b0f90d5d340fbef80233f33347
author Felipe Leme <felipeal@google.com> Mon Jun 13 10:08:59 2016 -0700
committer Felipe Leme <felipeal@google.com> Mon Jun 13 10:08:59 2016 -0700
tree 3b1332386e694174ee4a3c01ffec03c6376c1bfd
parent 194198f5c1f3f12a41ec9c5ad52a0e3a5812d1c1

Fixed nullptr check.

arg[0] is always the command being passed to execvp and hence it will
never be null; the real args start at arg[1]

BUG: 27832567
Change-Id: I3180eae857cf028315b1da7cbd38b40994ff98b0
Fixes: 29276781

cmds/dumpstate/utils.cpp

diff --git a/cmds/dumpstate/utils.cpp b/cmds/dumpstate/utils.cpp
index 992f0a9..0850468 100644
--- a/cmds/dumpstate/utils.cpp
+++ b/cmds/dumpstate/utils.cpp
@@ -1376,7 +1376,7 @@
 void format_args(const char* command, const char *args[], std::string *string) {
     LOG_ALWAYS_FATAL_IF(args == nullptr || command == nullptr);
     string->append(command);
-    if (args[0] == nullptr) return;
+    if (args[1] == nullptr) return;
     string->append(" ");
 
     for (int arg = 1; arg <= 1000; ++arg) {