]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUILD: ssl: can't build when using -DLISTEN_DEFAULT_CIPHERS
authorWilliam Lallemand <wlallemand@haproxy.com>
Fri, 17 Oct 2025 07:35:41 +0000 (09:35 +0200)
committerWilliam Lallemand <wlallemand@haproxy.com>
Fri, 17 Oct 2025 07:45:26 +0000 (09:45 +0200)
commitb74a437e576fa192b96c1f2bc90d67766376d751
treecf529a05d6aaa3a9868b08becf615783d5373db9
parent5b04a85bc7cd224ba352498b1424e1881f655a4c
BUILD: ssl: can't build when using -DLISTEN_DEFAULT_CIPHERS

Emeric reported that he can't build haproxy anymore since 9bc6a034
("BUG/MINOR: ssl: Free global_ssl structure contents during deinit").

    src/ssl_sock.c:7020:40: error: comparison with string literal results in unspecified behavior [-Werror=address]
     7020 |  if (global_ssl.listen_default_ciphers != LISTEN_DEFAULT_CIPHERS)
          |                                        ^~
    src/ssl_sock.c:7023:41: error: comparison with string literal results in unspecified behavior [-Werror=address]
     7023 |  if (global_ssl.connect_default_ciphers != CONNECT_DEFAULT_CIPHERS)
          |                                         ^~
    src/ssl_sock.c: At top level:

Indeed the mentionned patch is checking the pointer in order to free
something freeable, but that can't work because these constant are
strings literal which can be passed from the compiler and not pointers.

Also the test is not useful, because these strings are strdup() in
__ssl_sock_init, so they can be free directly.

Must be backported in every stable branches with 9bc6a034.
src/ssl_sock.c