From: Jérémie Galarneau Date: Mon, 22 Apr 2024 19:52:28 +0000 (-0400) Subject: lttng-ctl: Hide symbol introduced by fix X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=83f2539c5c084d853308993a05999016264be90e;p=lttng-tools.git lttng-ctl: Hide symbol introduced by fix Michael Jeanson noticed that f94318611 mistakenly introduces a new public symbol in liblttng-ctl.so. This change was not intended and is due to a bad backport of a fix initially developed against the master branch. The master branch (and upcoming version) of LTTng-tools is built with the -fvisibility=hidden. Hence, the initial version of the fix had no need to hide the utils_create_lock_file symbol. Since the supported stable releases (2.12 and 2.13) are not built with those options, that symbol has to be explicitly marked as hidden. Reported-by: Michael Jeanson Signed-off-by: Jérémie Galarneau Change-Id: I874538317617003eb1c58c2d3b7b0a2bdf905ef7 --- diff --git a/src/common/lockfile.cpp b/src/common/lockfile.cpp index 3e972fbac..aeafd85cd 100644 --- a/src/common/lockfile.cpp +++ b/src/common/lockfile.cpp @@ -64,6 +64,7 @@ static int lock_file(const char *filepath, int fd) #endif /* HAVE_FLOCK */ +LTTNG_HIDDEN int utils_create_lock_file(const char *filepath) { int ret, fd; diff --git a/src/common/lockfile.hpp b/src/common/lockfile.hpp index 1f8e01ddb..67bae3046 100644 --- a/src/common/lockfile.hpp +++ b/src/common/lockfile.hpp @@ -8,6 +8,8 @@ #ifndef COMMON_LOCKFILE_H #define COMMON_LOCKFILE_H +#include + /* * Create lock file to the given path and filename. * Returns the associated file descriptor, -1 on error. @@ -20,6 +22,7 @@ * forks+execve as the child process closes a file descriptor referencing * the file description of 'filepath'. */ +LTTNG_HIDDEN int utils_create_lock_file(const char *filepath); #endif /* COMMON_LOCKFILE_H */