commit 604d87414393cd07fb7341a8fdb4802b8de390be
author Katz Yamada <kyamada@codeaurora.org> Thu May 04 14:43:52 2017 -0700
committer Katz Yamada <kyamada@codeaurora.org> Mon May 08 16:29:05 2017 -0700
tree dd8c25f4450bec635aefb66fcd7c41018303ff66
parent 10ab9ccaaf7f24ce126fd0d7998f6164a5e5f871

Fix for buffer overrun crash at copying nmea string

Add zero clearing of allocated nmea buffer to ensure
the nmea string is null terminated.

Change-Id: Ie36010a7d3eca16dabb3067ae891a94e4b63b10c
CRs-Fixed: 2041933

core/SystemStatus.cpp

diff --git a/core/SystemStatus.cpp b/core/SystemStatus.cpp
index 12f97dd..f4f07f2 100644
--- a/core/SystemStatus.cpp
+++ b/core/SystemStatus.cpp
@@ -1396,7 +1396,7 @@
     }
 
     char buf[SystemStatusNmeaBase::NMEA_MAXSIZE + 1] = { 0 };
-    strlcpy(buf, data, (len < strlen(data))? len : strlen(data));
+    strlcpy(buf, data, sizeof(buf));
 
     pthread_mutex_lock(&mMutexSystemStatus);