Make LogMessage() handle strings with null bytes

'LOG(...) << std::string("a\0a", 3);' would cause the line to be
truncated, and no newline output.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=257093

Review URL: https://codereview.chromium.org/198103008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@257135 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 3 deletions

diff --git a/base/logging.cc b/base/logging.cc
index de88a86..0e66946 100644
--- a/base/logging.cc
+++ b/base/logging.cc
@@ -597,13 +597,13 @@ LogMessage::~LogMessage() {
     }
     __android_log_write(priority, "chromium", str_newline.c_str());
 #endif
-    fprintf(stderr, "%s", str_newline.c_str());
+    ignore_result(fwrite(str_newline.data(), str_newline.size(), 1, stderr));
     fflush(stderr);
   } else if (severity_ >= kAlwaysPrintErrorLevel) {
     // When we're only outputting to a log file, above a certain log level, we
     // should still output to stderr so that we can better detect and diagnose
     // problems with unit tests, especially on the buildbots.
-    fprintf(stderr, "%s", str_newline.c_str());
+    ignore_result(fwrite(str_newline.data(), str_newline.size(), 1, stderr));
     fflush(stderr);
   }
 
@@ -628,7 +628,8 @@ LogMessage::~LogMessage() {
                 &num_written,
                 NULL);
 #else
-      fprintf(log_file, "%s", str_newline.c_str());
+      ignore_result(fwrite(
+          str_newline.data(), str_newline.size(), 1, log_file));
       fflush(log_file);
 #endif
     }