commit 7eb0592ca18384c17c75e51652960146b2c312e6
author Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Fri Dec 20 00:42:29 2019 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Fri Dec 20 00:42:29 2019 +0000
tree 6b903701b376bdcda16b07f9b465640b6fae3636
parent 5f699186e97a5ee7f6a4bddb11173bf07247097d
parent 9a11a0000407c9bde1a92d356bdaed4068cbc0df

Merge "Fix error checking in append_string" am: e2e2b2d3d3 am: 33e2002540

Change-Id: Ia3c7b38e0400ebd045fa67cfa914f7880840e77d

libstats/socket/stats_event.c

diff --git a/libstats/socket/stats_event.c b/libstats/socket/stats_event.c
index 551b392..15039c6 100644
--- a/libstats/socket/stats_event.c
+++ b/libstats/socket/stats_event.c
@@ -141,7 +141,7 @@
 // Side-effect: modifies event->errors if buf is not properly null-terminated
 static void append_string(struct stats_event* event, const char* buf) {
     size_t size = strnlen(buf, MAX_EVENT_PAYLOAD);
-    if (event->errors) {
+    if (size == MAX_EVENT_PAYLOAD) {
         event->errors |= ERROR_STRING_NOT_NULL_TERMINATED;
         return;
     }