A bit more clean up for Linux upload crash dump....

chrome/app/breakpad_linux.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <fcntl.h>
 #include <sys/socket.h>
 #include <sys/uio.h>
 #include <unistd.h>
 
 #include <string>
@@ skipping 314 matching lines @@
     sys_setsid();
 
     // Leave one end of a pipe in the wget process and watch for it getting
     // closed by the wget process exiting.
     int fds[2];
     sys_pipe(fds);
 
     const pid_t child = sys_fork();
     if (child) {
       sys_close(fds[1]);
-      char buf[17];
-      HANDLE_EINTR(read(fds[0], buf, sizeof(buf) - 1));
-      buf[sizeof(buf) - 1] = 0;
-      static const char msg[] = "\nCrash dump id: ";
-      sys_write(2, msg, sizeof(msg) - 1);
-      sys_write(2, buf, my_strlen(buf));
-      sys_write(2, "\n", 1);
+      char id_buf[17];
+      const int len = HANDLE_EINTR(read(fds[0], id_buf, sizeof(id_buf) - 1));
+      if (len > 0) {
+        id_buf[len] = 0;
+        static const char msg[] = "\nCrash dump id: ";
+        sys_write(2, msg, sizeof(msg) - 1);
+        sys_write(2, id_buf, my_strlen(buf));
+        sys_write(2, "\n", 1);
+      }
       sys_unlink(filename);
       sys_unlink(buf);
       sys__exit(0);
     }
 
     sys_close(fds[0]);
     sys_dup2(fds[1], 3);
     static const char* const kWgetBinary = "/usr/bin/wget";
     const char* args[] = {
       kWgetBinary,
@@ skipping 125 matching lines @@
 
   // Determine the process type and take appropriate action.
   const CommandLine& parsed_command_line = *CommandLine::ForCurrentProcess();
   const std::wstring process_type =
       parsed_command_line.GetSwitchValue(switches::kProcessType);
   if (process_type.empty())
     EnableCrashDumping();
   else if (process_type == switches::kRendererProcess)
     EnableRendererCrashDumping();
 }