commit 033d4245d0f6f8a0690c50a629a2b219bb54775e
author Ruchir Rastogi <ruchirr@google.com> Wed Dec 18 13:43:05 2019 -0800
committer Ruchir Rastogi <ruchirr@google.com> Thu Dec 19 14:57:10 2019 -0800
tree d51d7aa3b4edad895f69c5a3e2ab486f8e0b1f84
parent e3f6bb1328f923ddbd41c87291f8862d10b05c15

Fix error checking in append_string

Test: m -j
Change-Id: Ifa806d5037acb17f83cbf45712e9795d6f56152f

libstats/socket/stats_event.c

diff --git a/libstats/socket/stats_event.c b/libstats/socket/stats_event.c
index 35081dc..cd4eb5f 100644
--- a/libstats/socket/stats_event.c
+++ b/libstats/socket/stats_event.c
@@ -142,7 +142,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;
     }