commit 36f0595547afd7134dc8cf875c839771aceeb4de
author Steven Moreland <smoreland@google.com> Fri Aug 02 10:13:57 2019 -0700
committer Steven Moreland <smoreland@google.com> Fri Aug 02 10:14:37 2019 -0700
tree 4f96f8e2fde57c373c4903696f41c253076cbde4
parent 87b6d86ac30e7aaa88a7adc0452cd66d88206d99

logcat: error on multiple PID argument

Since they are dropped.

Bug: N/A
Test: manual try w/ 1 and 2 --pid arguments
Change-Id: I5945c12dc080a8c046b8071106f7d87ecb2b018c

logcat/logcat.cpp

diff --git a/logcat/logcat.cpp b/logcat/logcat.cpp
index 6e38d95..6507711 100644
--- a/logcat/logcat.cpp
+++ b/logcat/logcat.cpp
@@ -899,6 +899,11 @@
             case 0:
                 // only long options
                 if (long_options[option_index].name == pid_str) {
+                    if (pid != 0) {
+                        logcat_panic(context, HELP_TRUE, "Only supports one PID argument.\n");
+                        goto exit;
+                    }
+
                     // ToDo: determine runtime PID_MAX?
                     if (!getSizeTArg(optarg, &pid, 1)) {
                         logcat_panic(context, HELP_TRUE, "%s %s out of range\n",