Home - Waterfall Grid T-Grid Console Builders Recent Builds Buildslaves Changesources - JSON API - About

Builder curl-threaded-solaris10-i386 Build #3619

Results:

Failed runtest

SourceStamp:

Projectcurl
Repositoryhttps://github.com/curl/curl.git
Branchmaster
Revision3bb5e58c105d7be450b667858d1b8e7ae3ded555
Got Revision3bb5e58c105d7be450b667858d1b8e7ae3ded555
Changes23 changes

BuildSlave:

unstable10x

Reason:

The SingleBranchScheduler scheduler named 'schedule-curl-threaded-solaris10-i386' triggered this build

Steps and Logfiles:

  1. git update ( 51 mins, 46 secs )
    1. stdio
  2. Runtest './tests/testcurl.pl --nogitpull ...' failed ( 3 hrs, 21 mins, 31 secs )
    1. stdio
    2. resultlog
  3. Mail result 'cat resultlog ...' ( 10 mins, 16 secs )
    1. stdio

Build Properties:

NameValueSource
branch master Build
builddir /export/home/buildbot/slave/curl-threaded-solaris10-i386 slave
buildername curl-threaded-solaris10-i386 Builder
buildnumber 3619 Build
codebase Build
got_revision 3bb5e58c105d7be450b667858d1b8e7ae3ded555 Git
osplatform I386 SetPropertyFromCommand Step
osrelease 10 SetPropertyFromCommand Step
project curl Build
repository https://github.com/curl/curl.git Build
revision 3bb5e58c105d7be450b667858d1b8e7ae3ded555 Build
scheduler schedule-curl-threaded-solaris10-i386 Scheduler
slavename unstable10x BuildSlave
workdir /export/home/buildbot/slave/curl-threaded-solaris10-i386 slave (deprecated)

Forced Build Properties:

NameLabelValue

Responsible Users:

  1. Daniel Stenberg
  2. Stefan Eissing
  3. Viktor Szakats

Timing:

StartMon Jul 28 18:56:46 2025
EndTue Jul 29 08:33:43 2025
Elapsed13 hrs, 36 mins, 57 secs

All Changes:

:

  1. Change #240932

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 10:32:13
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 5debe7cb34bb6f0ee094515c17b7662e386829fb

    Comments

    CURLOPT: drop redundant `long` casts
Also:
- CURLOPT_HSTS_CTRL.md: sync macro definitions with `curl/curl.h`.
  Perhaps it'd be better to delete copies like this?
- keep existing casts within the documentation to make sure it applies
  to older curl versions as well.
- CURLOPT_IPRESOLVE.md: re-add a long cast to man page, for consistency
  with the above.

Closes #17791

    Changed files

    • docs/examples/anyauthput.c
    • docs/examples/ghiper.c
    • docs/examples/hsts-preload.c
    • docs/examples/http3.c
    • docs/examples/imap-tls.c
    • docs/examples/pop3-tls.c
    • docs/examples/smtp-tls.c
    • docs/libcurl/opts/CURLOPT_HSTS_CTRL.md
    • docs/libcurl/opts/CURLOPT_HTTP_VERSION.md
    • docs/libcurl/opts/CURLOPT_IPRESOLVE.md
    • src/config2setopts.c
    • tests/client/hx_download.c
    • tests/client/hx_upload.c
    • tests/libtest/lib1511.c
    • tests/libtest/lib1513.c
    • tests/libtest/lib1555.c
    • tests/libtest/lib1568.c
    • tests/libtest/lib1593.c
    • tests/libtest/lib1599.c
    • tests/libtest/lib1662.c
    • tests/libtest/lib2309.c
    • tests/libtest/lib2502.c
    • tests/libtest/lib510.c
    • tests/libtest/lib540.c
    • tests/libtest/lib547.c
    • tests/libtest/lib552.c
    • tests/libtest/lib555.c
    • tests/libtest/lib579.c
    • tests/libtest/lib583.c
    • tests/libtest/lib590.c
    • tests/libtest/lib678.c
    • tests/libtest/lib694.c

  2. Change #240933

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 10:32:13
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 430f9b03fd2b11804e6ae6147efaefb1f5a9916d

    Comments

    CURLOPT: bump `CURLFTP*` enums to `long`, drop casts
This patch bumps the size of these constants from `int` to `long`, while
keeping their actual values the same. It may cause incompatibilities in
user code, requiring the bump of holder variables and/or adding casts:

- CURLFTP_CREATE_DIR
- CURLFTP_CREATE_DIR_NONE
- CURLFTP_CREATE_DIR_RETRY
- CURLFTPAUTH_DEFAULT
- CURLFTPAUTH_SSL
- CURLFTPAUTH_TLS
- CURLFTPMETHOD_DEFAULT
- CURLFTPMETHOD_MULTICWD
- CURLFTPMETHOD_NOCWD
- CURLFTPMETHOD_SINGLECWD
- CURLFTPSSL_CCC_ACTIVE
- CURLFTPSSL_CCC_NONE
- CURLFTPSSL_CCC_PASSIVE

Also:
- keep existing casts within the documentation to make sure it applies
  to older curl versions as well.

Closes #17797

    Changed files

    • docs/libcurl/opts/CURLOPT_FTP_CREATE_MISSING_DIRS.md
    • include/curl/curl.h
    • lib/ftp.c
    • lib/setopt.c
    • src/config2setopts.c
    • src/tool_cfgable.h
    • src/tool_paramhlp.c
    • src/tool_paramhlp.h
    • tests/libtest/lib539.c
    • tests/libtest/lib661.c

  3. Change #240941

    Category curl
    Changed by Stefan Eissing <stefanohnoyoudont@eissing.org>
    Changed at Mon 28 Jul 2025 10:53:51
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 44f53078914d1f827bfb3252276d58feb3ca82b5

    Comments

    multi: fix assert in multi_getsock()
Now that multi keeps the "dirty" bitset, the detection of possibly
stalling transfers needs to adapt. Before dirty, transfers needed
to expose a socket to poll or a timer to wait for.

Dirty transfer might no longer have a timer, but will run, so do
not need to report a socket. Adjust the assert condition.

Fixes #18046
Reported-by: Viktor Szakats
Closes #18051

    Changed files

    • lib/multi.c

  4. Change #240951

    Category curl
    Changed by Stefan Eissing <stefanohnoyoudont@eissing.org>
    Changed at Mon 28 Jul 2025 11:18:07
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 8e1d817cb36bca2ed08296c363d89b75c7779dc8

    Comments

    build: fix disable-verbose
Fix compile error when building with `--disable-verbose`.

Adjust pytest to skip when curl is not a debug build but needs
traces.

Follow-up to b453a447ce4e6aa2a352b196

Closes #18053

    Changed files

    • lib/connect.c
    • lib/curl_trc.c
    • lib/http2.c
    • lib/multi.c
    • tests/http/test_02_download.py
    • tests/http/test_10_proxy.py
    • tests/http/test_13_proxy_auth.py
    • tests/http/test_15_tracing.py
    • tests/http/test_17_ssl_use.py
    • tests/http/test_30_vsftpd.py
    • tests/http/test_31_vsftpds.py
    • tests/http/test_32_ftps_vsftpd.py

  5. Change #240958

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 11:28:15
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 58b9c6134bf8632442aa7d703aa8c7061604785e

    Comments

    cmake: omit linking duplicate/unnecessary libs to tests & examples
Before this patch we explicitly linked the full list of libcurl
dependency libs to tests and examples via `CURL_LIBS`. This was
redundant, because test and example code do not directly use these
dependency libs and for indirect use they are implicitly passed
via libcurl as needed. After this patch, tests and examples only link
explicitly to system libs (e.g. socket).

Also bringing it closer to how `./configure` does this.

Borrow the variable name `CURL_NETWORK_AND_TIME_LIBS` from
`./configure`. However, its content is not exactly the same. With cmake
it also holds `pthread`, but doesn't hold AmiSSL.

Closes #17696

    Changed files

    • CMakeLists.txt
    • docs/examples/CMakeLists.txt
    • tests/client/CMakeLists.txt
    • tests/libtest/CMakeLists.txt
    • tests/server/CMakeLists.txt

  6. Change #240977

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 12:43:01
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 2c90c3aac0a7a348db140702b5b7ff4b83642799

    Comments

    build: tidy up compiler definition for tests
- tests: merge cmake commands.
- tests: use `target_compile_definitions()`.
- tests/server: use generator expression for platform-specific macro.
- tests/unit: sync `Makefile.am` comment with cmake.
- tests/unit: merge two `AM_CPPFLAGS` lines to keep synced with cmake.
- tests: move macro definitions to `first.h` headers from build level.
  `CURL_NO_OLDIES`, `CURL_DISABLE_DEPRECATION`, `WITHOUT_LIBCURL`,
  `CURL_STATICLIB` (for servers).
  To share more logic.
  Pass `CURL_STATICLIB` in server on all platforms for simplicity.
  (On non-Windows, it's a no-op. It's already done like this with curlu
  and libcurltool.)

Also for lib:
- lib: merge commands.
- lib: sync macro order with tests (also in `Makefile.am`).

Closes #17768

    Changed files

    • lib/CMakeLists.txt
    • lib/Makefile.am
    • tests/client/CMakeLists.txt
    • tests/client/Makefile.am
    • tests/client/first.h
    • tests/libtest/CMakeLists.txt
    • tests/libtest/Makefile.am
    • tests/libtest/first.h
    • tests/server/CMakeLists.txt
    • tests/server/Makefile.am
    • tests/server/first.h
    • tests/tunit/CMakeLists.txt
    • tests/tunit/Makefile.am
    • tests/unit/CMakeLists.txt
    • tests/unit/Makefile.am

  7. Change #240980

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 12:43:02
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision a28f5f68b965119d9dd1ab6c2a2ccc66c6ed5d1f

    Comments

    build: bump minimum required mingw-w64 to v3.0 (from v1.0)
mingw-w64 3.0 was released on 2013-09-20. Offered by Debian jessie.

1.0 and 2.0 were released in 2011. It seems unlikely that many people
use them. The oldest downloadable toolchain (that I know of) comes with
3.0. Due to this, older versions weren't CI tested, and probably seldom
tested elsewhere. The last bugfix update for both 1.0 and 2.0 was
released in 2015.

curl can now assume availability of these 3.0 features/fixes:
- 64-bit file offsets.
- `ADDRESS_FAMILY` type.
- `__MINGW_PRINTF_FORMAT` macro. (in public curl headers)

Public curl headers keep supporting older mingw-w64 versions.

Fixes #17984
Closes #18010

    Changed files

    • CMake/win32-cache.cmake
    • CMakeLists.txt
    • docs/INSTALL.md
    • lib/curl_setup.h
    • lib/easy_lock.h

  8. Change #240981

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 12:43:02
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 9a68a86ea3626d5175d9dc9b521c969d80b7771d

    Comments

    memanalyze: fix warnings
Also fix possibly missing reallocated memory from 'Total allocated':
```
Use of uninitialized value $size in addition (+) at tests/memanalyze.pl line 240, <$fileh> line 4.
```
Ref: https://github.com/curl/curl/actions/runs/16565283280/job/46843800711?pr=18049#step:39:3834

Ref: https://github.com/curl/curl/actions/runs/16556860012/job/46819517495?pr=17927#step:39:156
Follow-up to fc98a630cfb27dd097b31b57aa59fb578b390063 #18048
Follow-up to 2ec54556d4e3f3ab551b5298adab0c703d85a463 #17877
Closes #18049

    Changed files

    • tests/memanalyze.pl

  9. Change #240995

    Category curl
    Changed by Stefan Eissing <stefanohnoyoudont@eissing.org>
    Changed at Mon 28 Jul 2025 12:57:51
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 366c589c815fbb7f368ca7c042da0d4693dff6ae

    Comments

    pytest: relax error check on test_07_22
Add code 56 to the list of accepted errors, as it varies
with backend and speediness of test run.

Closes #18050

    Changed files

    • tests/http/test_07_upload.py

  10. Change #240996

    Category curl
    Changed by Daniel Stenberg <danielohnoyoudont@haxx.se>
    Changed at Mon 28 Jul 2025 13:04:36
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 22d6ac0b65b19e3f13645f9e548dcae8599c88c4

    Comments

    RELEASE-NOTES: synced

    Changed files

    • RELEASE-NOTES

  11. Change #241001

    Category curl
    Changed by Daniel Stenberg <danielohnoyoudont@haxx.se>
    Changed at Mon 28 Jul 2025 14:02:52
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision f63bdea79028c30780b3450e5d444c84b63a5434

    Comments

    delta: drop the warnings+strict
They really don't do much good here and I rather do this than adding
'my' on 40+ lines.

Follow-up from 2ec54556d4e3f3ab55

Closes #18058

    Changed files

    • scripts/delta

  12. Change #241002

    Category curl
    Changed by Stefan Eissing <stefanohnoyoudont@eissing.org>
    Changed at Mon 28 Jul 2025 14:04:31
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 21e885eb39c72c9a4ad59432e5de8a9859934436

    Comments

    alpn: query filter
Add a connection filter query to obtained the negotiated ALPN
protocol to check in setup/protocols how the connection needs
to behave.

Remove the members `alpn` and `proxy_alpn` from `connectdata`.

Closes #17947

    Changed files

    • lib/cf-h2-proxy.c
    • lib/cf-https-connect.c
    • lib/cfilters.c
    • lib/cfilters.h
    • lib/http.c
    • lib/http_proxy.c
    • lib/socks.c
    • lib/url.c
    • lib/urldata.h
    • lib/vquic/curl_ngtcp2.c
    • lib/vquic/curl_osslq.c
    • lib/vquic/curl_quiche.c
    • lib/vtls/schannel.c
    • lib/vtls/vtls.c
    • tests/http/test_10_proxy.py
    • tests/http/test_13_proxy_auth.py

  13. Change #241008

    Category curl
    Changed by Daniel Stenberg <danielohnoyoudont@haxx.se>
    Changed at Mon 28 Jul 2025 14:17:40
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision e82c172e9f9caa9f26c09d87772a3a58d423f688

    Comments

    docs/cmdline-opts: the auth types are not mutually exclusive
They are booleans

URL: https://curl.se/mail/archive-2025-07/0019.html
Closes #18059

    Changed files

    • docs/cmdline-opts/anyauth.md
    • docs/cmdline-opts/basic.md
    • docs/cmdline-opts/digest.md
    • docs/cmdline-opts/negotiate.md
    • docs/cmdline-opts/ntlm.md
    • docs/cmdline-opts/proxy-anyauth.md
    • docs/cmdline-opts/proxy-basic.md
    • docs/cmdline-opts/proxy-digest.md
    • docs/cmdline-opts/proxy-ntlm.md

  14. Change #241010

    Category curl
    Changed by Daniel Stenberg <danielohnoyoudont@haxx.se>
    Changed at Mon 28 Jul 2025 14:18:47
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 0e492341283778c37cf190abd22424942902f34d

    Comments

    _PROTOCOLS.md: mention file:// is only for absolute paths
... when using curl.

Asked-by: Paul Gilmartin
URL: https://curl.se/mail/archive-2025-07/0018.html
Closes #18060

    Changed files

    • docs/cmdline-opts/_PROTOCOLS.md

  15. Change #241015

    Category curl
    Changed by Stefan Eissing <stefanohnoyoudont@eissing.org>
    Changed at Mon 28 Jul 2025 14:57:38
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 6845533e24f6febc7a006224b8991de5da328667

    Comments

    curl: add long option '--out-null'
Add a new commandline option --out-null that discards all
response bytes into the void. Replaces non-portable use of
'-o /dev/null' with more efficiency.

Feature added in 8.16.0

Closes #17800

    Changed files

    • docs/cmdline-opts/Makefile.inc
    • docs/cmdline-opts/out-null.md
    • docs/cmdline-opts/output.md
    • docs/options-in-versions
    • src/tool_cb_wrt.c
    • src/tool_getparam.c
    • src/tool_getparam.h
    • src/tool_listhelp.c
    • src/tool_operate.c
    • src/tool_sdecls.h
    • tests/data/Makefile.am
    • tests/data/test756
    • tests/http/testenv/curl.py

  16. Change #241031

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 17:29:26
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 975ab36531adf01970814cd81049bcc508c89c61

    Comments

    lib517: use `LL` 64-bit literals & re-enable a test case (`time_t`)
Suffix two 64-bit `time_t` test literals with `LL` to make them compile
with mingw-w64 x86_64 in C89 (the default) mode. Possibly other old gcc
compilers are affected (e.g. mips gcc 4.9.4, power gcc 15.1.0), but
could not pinpoint the exact rules. This also fixes a compiler warning
and test failure with MSVC, allowing to re-enable a disabled test case.

`LL` is not C89, but used in the code before this patch, which tells
it's safe to use.

Also display expected / actual timestamp values as `curl_off_t` instead
of `long`, making them work with 64-bit timestamps.

This was triggered by this issue seen while testing mingw-w64 gcc 4.8.1:
```
tests/libtest/lib517.c:147:5: error: this decimal constant is unsigned only in ISO C90
     {"Sun, 06 Nov 2044 08:49:37 GMT", (time_t) 2362034977 },
     ^
```
Ref: https://github.com/curl/curl/actions/runs/16540378828/job/46780712313?pr=18010#step:12:32

Closes #18032

    Changed files

    • tests/libtest/lib517.c

  17. Change #241032

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 17:29:26
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision b1df1d38afce5b2d9de3a60eacdc041c5d575498

    Comments

    delta: fix warnings, fix for non-GNU `date` tool
It makes the script run on BSD-like envs.

Follow-up to f63bdea79028c30780b3450e5d444c84b63a5434 #18058
Follow-up to 2ec54556d4e3f3ab551b5298adab0c703d85a463 #17877

Closes #18061

    Changed files

    • scripts/delta

  18. Change #241033

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 17:29:26
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 577ec0941235e52dcb39765c02c25444766ac2de

    Comments

    contrithanks: fix for BSD `sed` tool
Fixing on macOS, and possibly other BSDs:
```
sed: 83: ./docs/THANKS-filter: RE error: illegal byte sequence
```
Where line 83 contains `\xED`.

Switch to raw encoding to avoid `sed` evaluating the stream of bytes.

Ref: #18061

Closes #18062

    Changed files

    • scripts/contrithanks.sh

  19. Change #241034

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 17:29:57
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 81f5991e14abe71ccee22e9576dc447e8c3418f4

    Comments

    lib1560: replace an `int` with `bool`
Follow-up to 7d1ca2e7e1a3b9d3ba70f587f6bc5e83f905afd5 #17933

Closes #18064

    Changed files

    • tests/libtest/lib1560.c

  20. Change #241035

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 17:30:36
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision d45b85d79198a0259123923a6aa186ded2aaca6c

    Comments

    CURLOPT: bump `CURLALTSVC_*` macros to `long`
This patch bumps the size of these macros from `int` to `long`, while
keeping their actual values the same. It may cause incompatibilities in
user code, requiring the bump of holder variables and/or adding casts:

- CURLALTSVC_H1
- CURLALTSVC_H2
- CURLALTSVC_H3
- CURLALTSVC_READONLYFILE

Also:
- keep existing cast within the documentation to make sure it applies
  to older curl versions as well.

Closes #18063

    Changed files

    • include/curl/curl.h

  21. Change #241036

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 17:30:46
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 1a12663d06d9c42fd40a8655d5247f7afeeb8e1e

    Comments

    CURLOPT: bump `CURLPROXY_*` enums to `long`, drop casts
This patch bumps the size of these macros from `int` to `long`, while
keeping their actual values the same. It may cause incompatibilities in
user code, requiring the bump of holder variables and/or adding casts:

- CURLPROXY_HTTP
- CURLPROXY_HTTP_1_0
- CURLPROXY_HTTPS
- CURLPROXY_HTTPS2
- CURLPROXY_SOCKS4
- CURLPROXY_SOCKS4A
- CURLPROXY_SOCKS5
- CURLPROXY_SOCKS5_HOSTNAME

Also:
- keep existing cast within the documentation to make sure it applies
  to older curl versions as well.

Closes #18054

    Changed files

    • include/curl/curl.h
    • lib/setopt.c
    • lib/url.c
    • lib/urldata.h
    • tests/libtest/lib1525.c
    • tests/libtest/lib1526.c
    • tests/libtest/lib1527.c
    • tests/libtest/lib1528.c
    • tests/libtest/lib1529.c
    • tests/libtest/lib564.c

  22. Change #241037

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 17:31:33
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 113f6aacd4ae4eaa25fca91932013425fca3788d

    Comments

    CURLOPT: bump `CURLHEADER_*` macros to `long`, drop casts
This patch bumps the size of these macros from `int` to `long`, while
keeping their actual values the same. It may cause incompatibilities in
user code, requiring the bump of holder variables and/or adding casts:

- CURLHEADER_SEPARATE
- CURLHEADER_UNIFIED

Also:
- keep existing cast within the documentation to make sure it applies
  to older curl versions as well.

Closes #18055

    Changed files

    • include/curl/curl.h
    • tests/libtest/lib1525.c
    • tests/libtest/lib1526.c
    • tests/libtest/lib1527.c
    • tests/libtest/lib1528.c

  23. Change #241039

    Category curl
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Mon 28 Jul 2025 17:45:04
    Repository https://github.com/curl/curl.git
    Project curl
    Branch master
    Revision 3bb5e58c105d7be450b667858d1b8e7ae3ded555

    Comments

    memory: make function overrides work reliably in unity builds
Fixing:
- HTTPS-RR builds with c-ares and Linux MUSL.
- curl-for-win minimal builds with Linux MUSL.

It should fix all other kinds of entaglement between curl's redefintions
of system symbols and system (or 3rd-party) headers sensitive to that.

It also syncs memory override behavior between unity & non-unity builds,
thus reducing build variations.

The idea is to define and declare everything once in `curl_setup.h`,
without overriding any system symbols with curl ones yet. Then, like
before this patch, override them, if necessary, in each source file via
`curl_memory.h` and `memdebug.h`, after including system headers.
To ensure a clean slate with no overrides at the beginning of each
source file, reset all of them unconditionally at the end of
`curl_setup.h`, by including `curl_mem_undef.h`. (This assumes
`curl_setup.h` is always included first, which is already the case
throughout the codebase.)

`curl_mem_undef.h` can also be included explicitly wherever overrides
are causing problems. E.g. in tests which use unity-style builds and
a previously included `curl_memory.h`/`memdebug.h` can be spilling into
other source files.

The simplified role of the two override headers:
- `curl_memory.h`: overrides system memory allocator functions to
  libcurl ones, when memory tracing (aka `CURLDEBUG`) is disabled.
- `memdebug.h`: overrides system memory allocator and some other
  functions to curl debug functions, when memory tracing is enabled.

Changed made in this patch, step-by-step:
- curl_memory.h: move allocator typedefs and protos to `curl_setup.h`.
- memdebug.h: move `ALLOC_*` macros to `curl_setup.h`.
- memdebug.h: move allocator protos to `curl_setup.h`.
- memdebug.h: move `Curl_safefree()` macro to `curl_setup.h`.
  (it's a regular macro, with a one-time, global, definition.)
- curl_memory.h: move system symbol undefs to a new, separate header:
  `curl_mem_undef.h`.
- curl_setup.h: include `curl_mem_undef.h` at the end, unconditionally,
  to reset system symbol macros after each inclusion.
- handle `sclose()` and `fake_sclose()` in `curl_setup.h`. They are not
  system symbols, a one-time definition does the job.

Also:
- GHA/linux: enable unity mode for the HTTP-RR c-ares MUSL job.
  Follow-up to 17ab4d62e62bd3bfb6dbf6cb51460629f21c03e7 #16413

That said, I'd still find it better to avoid redefining system macros.
To communicate clearly the fact that they are not the original system
calls and they do behave differently. And, it would allow dropping the
undef/redef dance in each source file, and maintaining the logic with
it. The "last #include files should be in this order" comments in each
source would also become unnecessary. Also the trick of using
`(func)` (or interim macros) to call the non-overridden function where
required. This method works for printf and most everything else already.
For `_tcsdup`, socket and fopen functions this could work without
disturbing the codebase much.

Ref: #16428 (clean reboot of)

Closes #17827

    Changed files

    • .github/workflows/linux.yml
    • lib/Makefile.inc
    • lib/curl_mem_undef.h
    • lib/curl_memory.h
    • lib/curl_setup.h
    • lib/curl_setup_once.h
    • lib/memdebug.h
BuildBot (0.8.14) working for the OpenCSW Buildbot project.
Page built: Sat 02 Aug 2025 07:16:16 (CEST)