lpdump: log warning message.

Warning messages were placed in the output stream as well.
Put them in logcat so that stdout is always a valid JSON
string.

Test: lpdump --json on walleye
Bug: 126233777
Change-Id: If313eeaee9c347c498818f53decdf092328f142c

1 files changed, 7 insertions, 2 deletions

diff --git a/partition_tools/lpdumpd.cc b/partition_tools/lpdumpd.cc
index 83ec60ed..6110f337 100644
--- a/partition_tools/lpdumpd.cc
+++ b/partition_tools/lpdumpd.cc
@@ -50,12 +50,17 @@ class Lpdump : public BnLpdump {
         }
         LOG(DEBUG) << "Dumping with args: " << base::Join(args, " ");
         std::stringstream output;
-        int ret = LpdumpMain((int)m_args.size(), argv, output, output);
+        std::stringstream error;
+        int ret = LpdumpMain((int)m_args.size(), argv, output, error);
+        std::string error_str = error.str();
         if (ret == 0) {
+            if (!error_str.empty()) {
+                LOG(WARNING) << error_str;
+            }
             *aidl_return = output.str();
             return Status::ok();
         } else {
-            return Status::fromServiceSpecificError(ret, output.str().c_str());
+            return Status::fromServiceSpecificError(ret, error_str.c_str());
         }
     }
 };