From 34f6fce335fcabfbff475a92f7d8724e186478db Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=A9mie=20Galarneau?= Date: Thu, 14 Apr 2022 17:36:54 -0400 Subject: [PATCH] Fix: liblttng-ctl: non-packed structure used for tracker serialization MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Using unpacked structures in liblttng-ctl's protocol can cause issues when mixing sessiond and client of different bitness. In this specific case I doubt it causes a problem, but it could rightfully do on some architectures. Signed-off-by: Jérémie Galarneau Change-Id: Ie17096a55a4b7508c604e184cae877b83df6e451 --- src/common/tracker.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/common/tracker.c b/src/common/tracker.c index 1c50d9d79..006535046 100644 --- a/src/common/tracker.c +++ b/src/common/tracker.c @@ -20,7 +20,7 @@ struct process_attr_tracker_values_comm_header { uint32_t count; -}; +} LTTNG_PACKED; struct process_attr_tracker_value_comm { /* enum lttng_process_attr_value_type */ @@ -30,7 +30,7 @@ struct process_attr_tracker_value_comm { /* Includes the '\0' terminator. */ uint32_t name_len; } value; -}; +} LTTNG_PACKED; #define GET_INTEGRAL_COMM_VALUE(value_ptr, as_type) \ ((as_type)(is_signed(as_type) ? (value_ptr)->u._signed : \ -- 2.34.1