commit 9a11a0000407c9bde1a92d356bdaed4068cbc0df
author Ruchir Rastogi <ruchirr@google.com> Thu Dec 19 16:41:42 2019 -0800
committer android-build-merger <android-build-merger@google.com> Thu Dec 19 16:41:42 2019 -0800
tree 10ca90ff8663f59b727789fbb859e99bea5df607
parent 39ad5235c0954a1c4e2905797bd129b5b1b33ea3
parent 187a26c6d093d1a7e34e173ee2386f6227dd8135

Merge "Fix error checking in append_string"
am: e2e2b2d3d3

Change-Id: Ica7e466b74db7f7927ec9855b8a119b454cbc996

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;
     }