Merge \"Redirect bugreport stderr to host\'s stderr.\" into nyc-dev
am: 6d349051ce
Change-Id: I7ed56de80ff6ac698fb62820f24fca940d7c0905
diff --git a/adb/commandline.cpp b/adb/commandline.cpp
index a926fe5..1f7dccb 100644
--- a/adb/commandline.cpp
+++ b/adb/commandline.cpp
@@ -293,11 +293,16 @@
// stdout/stderr are routed independently and the remote exit code will be
// returned.
// if |output| is non-null, stdout will be appended to it instead.
-static int read_and_dump(int fd, bool use_shell_protocol=false, std::string* output=nullptr) {
+// if |err| is non-null, stderr will be appended to it instead.
+static int read_and_dump(int fd, bool use_shell_protocol=false, std::string* output=nullptr,
+ std::string* err=nullptr) {
int exit_code = 0;
+ if (fd < 0) return exit_code;
+
std::unique_ptr<ShellProtocol> protocol;
int length = 0;
FILE* outfile = stdout;
+ std::string* outstring = output;
char raw_buffer[BUFSIZ];
char* buffer_ptr = raw_buffer;
@@ -310,7 +315,7 @@
buffer_ptr = protocol->data();
}
- while (fd >= 0) {
+ while (true) {
if (use_shell_protocol) {
if (!protocol->Read()) {
break;
@@ -318,9 +323,11 @@
switch (protocol->id()) {
case ShellProtocol::kIdStdout:
outfile = stdout;
+ outstring = output;
break;
case ShellProtocol::kIdStderr:
outfile = stderr;
+ outstring = err;
break;
case ShellProtocol::kIdExit:
exit_code = protocol->data()[0];
@@ -338,11 +345,11 @@
}
}
- if (output == nullptr) {
+ if (outstring == nullptr) {
fwrite(buffer_ptr, 1, length, outfile);
fflush(outfile);
} else {
- output->append(buffer_ptr, length);
+ outstring->append(buffer_ptr, length);
}
}
@@ -1114,7 +1121,8 @@
static int send_shell_command(TransportType transport_type, const char* serial,
const std::string& command,
bool disable_shell_protocol,
- std::string* output=nullptr) {
+ std::string* output=nullptr,
+ std::string* err=nullptr) {
int fd;
bool use_shell_protocol = false;
@@ -1149,7 +1157,7 @@
}
}
- int exit_code = read_and_dump(fd, use_shell_protocol, output);
+ int exit_code = read_and_dump(fd, use_shell_protocol, output, err);
if (adb_close(fd) < 0) {
PLOG(ERROR) << "failure closing FD " << fd;
@@ -1160,8 +1168,7 @@
static int bugreport(TransportType transport_type, const char* serial, int argc,
const char** argv) {
- // No need for shell protocol with bugreport, always disable for simplicity.
- if (argc == 1) return send_shell_command(transport_type, serial, "bugreport", true);
+ if (argc == 1) return send_shell_command(transport_type, serial, "bugreport", false);
if (argc != 2) return usage();
// Zipped bugreport option - will call 'bugreportz', which prints the location of the generated
@@ -1175,7 +1182,7 @@
fprintf(stderr, "Bugreport is in progress and it could take minutes to complete.\n"
"Please be patient and do not cancel or disconnect your device until it completes.\n");
- int status = send_shell_command(transport_type, serial, "bugreportz", true, &output);
+ int status = send_shell_command(transport_type, serial, "bugreportz", false, &output, nullptr);
if (status != 0 || output.empty()) return status;
output = android::base::Trim(output);
@@ -1190,10 +1197,10 @@
}
if (android::base::StartsWith(output, BUGZ_FAIL_PREFIX)) {
const char* error_message = &output[strlen(BUGZ_FAIL_PREFIX)];
- fprintf(stderr, "device failed to take a zipped bugreport: %s\n", error_message);
+ fprintf(stderr, "Device failed to take a zipped bugreport: %s\n", error_message);
return -1;
}
- fprintf(stderr, "unexpected string (%s) returned by bugreportz, "
+ fprintf(stderr, "Unexpected string (%s) returned by bugreportz, "
"device probably does not support -z option\n", output.c_str());
return -1;
}