From: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Date: Wed, 16 Mar 2022 15:56:21 +0000 (-0400)
Subject: Fix: event: erroneous bound check on perf counter name size
X-Git-Tag: v2.12.9~4
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=420e125556b416f84193298c239410018a228697;p=lttng-tools.git

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.c b/src/common/event.c
index c52c4ee5a..531c73b9f 100644
--- a/src/common/event.c
+++ b/src/common/event.c
@@ -1093,7 +1093,8 @@ static ssize_t lttng_event_context_perf_counter_populate_from_buffer(
 			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;