[Tracing] Minor bugs fix.

src/libplatform/tracing/trace-object.cc

 // Copyright 2016 the V8 project 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 "include/libplatform/v8-tracing.h"
 
 #include "base/trace_event/common/trace_event_common.h"
 #include "src/base/platform/platform.h"
 #include "src/base/platform/time.h"
 
 namespace v8 {
 namespace platform {
 namespace tracing {
 
 // We perform checks for NULL strings since it is possible that a string arg
 // value is NULL.
 V8_INLINE static size_t GetAllocLength(const char* str) {
   return str ? strlen(str) + 1 : 0;
 }
 
 // Copies |*member| into |*buffer|, sets |*member| to point to this new
 // location, and then advances |*buffer| by the amount written.
 V8_INLINE static void CopyTraceObjectParameter(char** buffer,
                                                const char** member,
                                                const char* end) {

[Yang, 2016/08/10 19:56:57]

is the last parameter still necessary?

   if (*member) {
-    strncpy(*buffer, *member, end - *buffer);
+    size_t length = strlen(*member) + 1;
+    strncpy(*buffer, *member, length);
     *member = *buffer;
-    *buffer += strlen(*member) + 1;
+    *buffer += length;
   }
 }
 
 void TraceObject::Initialize(char phase, const uint8_t* category_enabled_flag,
                              const char* name, const char* scope, uint64_t id,
                              uint64_t bind_id, int num_args,
                              const char** arg_names, const uint8_t* arg_types,
                              const uint64_t* arg_values, unsigned int flags) {
   pid_ = base::OS::GetCurrentProcessId();
   tid_ = base::OS::GetCurrentThreadId();
@@ skipping 82 matching lines @@
   flags_ = flags;
   ts_ = ts;
   tts_ = tts;
   duration_ = duration;
   cpu_duration_ = cpu_duration;
 }
 
 }  // namespace tracing
 }  // namespace platform
 }  // namespace v8