summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Hans Boehm <hboehm@google.com> 2022-03-09 11:46:04 -0800
committer Hans Boehm <hboehm@google.com> 2022-03-09 11:46:04 -0800
commitedd99c3a7f22e2cb0aab16ec64647a16e9b06359 (patch)
treeca7ff8fed88566596262dbac52387e4f07b01776
parent6b8b9efad1b8977df5acf7ac3fa589fd9216fded (diff)
Correct off-by-one error in SetThreadName
This truncated thread names to one fewer character than was intended. Test: Build and boot AOSP. Change-Id: I38f56b473e70e1baf0b810944a9ccd275f272bef
-rw-r--r--core/jni/com_android_internal_os_Zygote.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/core/jni/com_android_internal_os_Zygote.cpp b/core/jni/com_android_internal_os_Zygote.cpp
index 597167026d19..5023927c2560 100644
--- a/core/jni/com_android_internal_os_Zygote.cpp
+++ b/core/jni/com_android_internal_os_Zygote.cpp
@@ -895,7 +895,7 @@ void SetThreadName(const std::string& thread_name) {
// pthread_setname_np fails rather than truncating long strings.
char buf[16]; // MAX_TASK_COMM_LEN=16 is hard-coded into bionic
- strlcpy(buf, name_start_ptr, sizeof(buf) - 1);
+ strlcpy(buf, name_start_ptr, sizeof(buf));
errno = pthread_setname_np(pthread_self(), buf);
if (errno != 0) {
ALOGW("Unable to set the name of current thread to '%s': %s", buf, strerror(errno));