mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-09-20 21:31:28 +02:00
Code Issues Projects Releases Wiki Activity

BUILD: systemd: fix usage of reserved name "sun" in the address field

Browse Source 
systemd.c doesn't build on Solaris / Illumos because it uses "sun" as
the field name in a structure, while "sun" is the name of the macro
used to detect Solaris:

  src/systemd.c: In function 'sd_notify':
  src/systemd.c:43:22: error: expected identifier or '(' before numeric constant
     struct sockaddr_un sun;
                        ^
  src/systemd.c:44:2: warning: no semicolon at end of struct or union
    } socket_addr = {
    ^

Admittedly, the OS could have instead defined "sun" to itself to avoid
this. Any other name will work, let's just use "ux" for the short form
of "unix".

The problem appeared in 3.0-dev with commit aa3632962f ("MEDIUM:
mworker: get rid of libsystemd"), though by then this file was only
built when USE_SYSTEMD was set, which was not the case for non-linux
platforms. However since 3.1-dev14 with commit 15845247db ("MEDIUM:
mworker: remove USE_SYSTEMD requirement for -Ws"), all platforms
now build this file.

No backport is needed even though it will not hurt to have it in 3.0
for completeness.
This commit is contained in:
Willy Tarreau 2024-11-25 08:04:09 +01:00
parent a941c92c12
commit f0548302bb
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/systemd.c
View File

@ -39,9 +39,9 @@ int sd_notify(int unset_environment, const char *message)
{
union sockaddr_union {
struct sockaddr sa;
struct sockaddr_un sun;
struct sockaddr_un ux;
} socket_addr = {
.sun.sun_family = AF_UNIX,
.ux.sun_family = AF_UNIX,
};
int ret = 1;
int fd = -1;
@ -77,16 +77,16 @@ int sd_notify(int unset_environment, const char *message)
path_length = strlen(socket_path);
/* Ensure there is room for NUL byte */
if (path_length >= sizeof(socket_addr.sun.sun_path)) {
if (path_length >= sizeof(socket_addr.ux.sun_path)) {
ret = -E2BIG;
goto end;
}
memcpy(socket_addr.sun.sun_path, socket_path, path_length);
memcpy(socket_addr.ux.sun_path, socket_path, path_length);
/* Support for abstract socket */
if (socket_addr.sun.sun_path[0] == '@')
socket_addr.sun.sun_path[0] = 0;
if (socket_addr.ux.sun_path[0] == '@')
socket_addr.ux.sun_path[0] = 0;
fd = socket(AF_UNIX, SOCK_DGRAM, 0);
if (fd < 0) {