Fix: event: erroneous bound check on perf counter name size

The wrong size if used when initializing a perf counter name from a
payload. The destination size must be used to prevent out of bound
writes.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Change-Id: I8ea41d30815bd2c02bb2ad8b01e8cecd2d6549a8

diff --git a/src/common/event.cpp b/src/common/event.cpp
index 7946d320975a08cdf54bc9bbd0d9a370f4e37687..601c6ac1fa5f1d2de8c13790102b97c994ab820c 100644 (file)
--- a/src/common/event.cpp
+++ b/src/common/event.cpp
@@ -1041,7 +1041,8 @@ static ssize_t lttng_event_context_perf_counter_populate_from_payload(
                        goto end;
                }
 
-               ret = lttng_strncpy(event_ctx->u.perf_counter.name, name, name_len);
+               ret = lttng_strncpy(event_ctx->u.perf_counter.name, name,
+                               sizeof(event_ctx->u.perf_counter.name));
                if (ret) {
                        consumed = -1;
                        goto end;