Fix: sessiond: missing include causes multiple conflicting definitions
authorJérémie Galarneau <jeremie.galarneau@efficios.com>
Wed, 24 Jul 2024 17:14:44 +0000 (17:14 +0000)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Tue, 30 Jul 2024 01:26:51 +0000 (01:26 +0000)
clang-tidy reports that
/root/lttng-tools/src/bin/lttng-sessiond/ust-ctl-internal.hpp:346:3: error: redefinition of 'LTTNG_PACKED' with a different type: 'struct lttng_ust_ctl_integer_type' vs 'struct lttng_ust_ctl_consumer_channel_attr' [clang-diagnostic-error]
  346 | } LTTNG_PACKED;

Indeed, that header is missing macros.hpp causing LTTNG_PACKED
to be misinterpreted.

Change-Id: I9a0ec916e7ad1c4d926f4d9a33dd6a85fb5b809c
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
src/bin/lttng-sessiond/ust-ctl-internal.hpp

index 79e07d9e81f149d9d9b7b702ac03411d3026ee9d..bb2b318680668ab0d56a03e5bf5b7467a5faffa9 100644 (file)
@@ -11,6 +11,8 @@
 
 #include "lttng-ust-abi.hpp"
 
+#include <common/macros.hpp>
+
 #include <limits.h>
 #include <sys/types.h>
 
This page took 0.02533 seconds and 4 git commands to generate.