Builder curl-threaded-solaris10-sparc Build #3482

Results:

Failed runtest

SourceStamp:

BuildSlave:

Reason:

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

Steps and Logfiles:

1. git update ( 33 secs )
   1. stdio
2. Runtest './tests/testcurl.pl --nogitpull ...' failed ( 5 hrs, 55 mins, 6 secs )
   1. stdio
   2. resultlog
3. Mail result 'cat resultlog ...' ( 35 secs )
   1. stdio

Build Properties:

Forced Build Properties:

Responsible Users:

1. Viktor Szakats

   commitohnoyoudont@vsz.me

Timing:

All Changes:

:

1. Change #258024

   Category	curl
   Changed by	Viktor Szakats <commitohnoyoudont@vsz.me>
   Changed at	Wed 18 Feb 2026 22:04:45
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	633ec719d5ae71f250156dcc06c264666c9a393a

   Comments

   curl_ntlm_core: merge two `#if` blocks
   Cherry-picked from #20593
   
   Closes #20620

   Changed files

   • lib/curl_ntlm_core.c

2. Change #258028

   Category	curl
   Changed by	Viktor Szakats <commitohnoyoudont@vsz.me>
   Changed at	Thu 19 Feb 2026 00:00:48
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	5fa5cb382560316a55f0954f1e8cebdbd6568cfb

   Comments

   build: fix `-Wunused-macros` warnings, and related tidy-ups
   - fix internal macro `AN_APPLE_OS` reused between sources without
     resetting it. It may potentially have left the system sha256
     function unused.
   - fix to define `WOLFSSL_OPTIONS_IGNORE_SYS` so that it always applies
     to wolfSSL headers, also during feature detection.
   - md4, md5, sha256: simplify fallback logic.
   - delete 20+ unused macros.
   - scope or move macros to avoid `-Wunused-macros` warnings.
   - examples: delete unused code.
   
   The warning detects macros defined but not used within the same C
   source. It does not warn for macros defined in headers. It also works
   with unity builds, but to a lesser extent.
   
   Closes #20593

   Changed files

   • CMakeLists.txt
   • docs/examples/log_failed_transfers.c
   • docs/examples/synctime.c
   • lib/asyn-ares.c
   • lib/cf-socket.c
   • lib/curl_ntlm_core.c
   • lib/ftp.c
   • lib/http.c
   • lib/http2.c
   • lib/md4.c
   • lib/md5.c
   • lib/parsedate.c
   • lib/progress.c
   • lib/setopt.c
   • lib/sha256.c
   • lib/url.c
   • lib/vauth/ntlm.c
   • lib/vquic/curl_ngtcp2.c
   • lib/vquic/curl_quiche.c
   • lib/vquic/vquic.c
   • lib/vquic/vquic_int.h
   • lib/vtls/openssl.c
   • lib/vtls/wolfssl.c
   • lib/vtls/x509asn1.c
   • lib/ws.c
   • m4/curl-wolfssl.m4
   • src/tool_doswin.c
   • src/var.c

3. Change #258030

   Category	curl
   Changed by	Viktor Szakats <commitohnoyoudont@vsz.me>
   Changed at	Thu 19 Feb 2026 00:02:11
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	4497dbd9ac4a31b0056a9179d500676ebe091de5

   Comments

   clang-tidy: fixes and improvements
   Fix bigger and smaller kinks in how clang-tidy is configured and used.
   Sync behavior more between autotools and cmake, lib/src and tests. Bump
   clang-tidy minimum version and prepare logic to allow using clang-tidy
   to a fuller extent.
   
   - move clang-tidy settings from builds to a new `.clang-tidy.yml`.
     To make it easy to see and edit checks at one place. Also to allow
     using the `--checks=` option internally to silence tests-specific
     checks. (clang-tidy does not support multiple `--check=` options via
     the command-line.)
     Use explicit `--config-file=` option to point to the configuration.
   - .clang-tidy.yml: link to documentation.
   - suppress `clang-diagnostic-nullability-extension` due to a false
     positive in libtests with `CURL_WERROR=ON` and `PICKY_COMPILER=OFF`.
   - .clang-tidy.yml: enable `portability-*`, `misc-const-correctness`.
   - drop `--quiet` clang-tidy option by default to make its working a bit
     more transparent. The extra output is minimial.
   - consistently use double-dashes in clang-tidy command-line options.
     Supported by clang-tidy 9.0.0+ (2019-09-19). Before this patch single
     and double were used arbitrarily.
   - src/tool_parsecfg: silence false positive `clang-analyzer-unix.Stream`.
     Seen with clang 18 + clang-tidy 19 and 20 (only with autotools.)
   - INTERNALS: require clang-tidy 14.0.0+. For the `--config-file` option.
   - INTERNALS: recommend clang-tidy 19.1.0+, to avoid bogus
     `clang-analyzer-valist.Uninitialized` warnings. (bug details below)
   
   autotools:
   
   - allow configuring the clang-tidy tool via `CLANG_TIDY` env.
     Also to use in GHA to point to a suffixed clang-tody tool.
   - fix to pass CFLAGS to lib, src sources.
     (keep omitting them when using a non-clang compiler.)
   - fix to pass `--warnings-as-errors=*` in quotes to avoid globbing.
   
   cmake:
   
   - fix to not pass an empty `-I` to clang-tidy.
   - fix to pass CFLAGS (picky warnings) to clang-tidy for test sources.
     (keep omitting them when using a non-clang compiler.)
   - fix to disable `clang-diagnostic-unused-function` for test sources.
     (tests have static entry points, which trigger this check when
     checking them as individidual sources.)
   - fix forwarding `CURL_CLANG_TIDYFLAGS` to clang-tidy.
   - force disable picky warnings when running clang-tidy with a non-clang
     compiler. To not pass these flags when checking lib and src.
   
   CI:
   
   - GHA/linux: avoid clang-tidy bug by upgrading to v19, and drop the
     workaround.
   - GHA/linux: switch to clang from gcc in the clang-tidy job. Using gcc
     doesn't allow passing CFLAGS to clang-tidy, making it less effective.
     (My guess this was one factor contributing to this job often missing
     to find certain issues compared to GHA/macos.)
   
   I recomment using clang-tidy with a clang compiler, preferably the same
   version or one that's compatible. Other cases are best effort, and may
   fail if a C flag is passed to clang-tidy that it does not understand.
   Picky warnings are mostly omitted when using a non-clang compiler,
   reducing its usefulness.
   
   Details and reproducer for the v18 (and earlier) clang-tidy bug,
   previously affecting the GHA/linux job:
   
   clang-tidy <=18 emits false warnings way when passing multiple C sources
   at once (as done with autotools):
   
   ```sh
   cat > src1.c <<EOF
   #include <string.h>
   static void dummy(void *p) { memcmp(p, p, 0); }
   EOF
   
   cat > src2.c <<EOF
   #include <stdarg.h>
   void vafunc(int option, ...)
   {
     va_list param;
     va_start(param, option);
     if(option)
       (void)va_arg(param, int);
     va_end(param);
   }
   EOF
   
   /opt/homebrew/opt/llvm@18/bin/clang-tidy --checks=clang-analyzer-valist.Uninitialized src1.c src2.c
   
   # src2.c:7:11: warning: va_arg() is called on an uninitialized va_list [clang-analyzer-valist.Uninitialized]
   ```
   
   Follow-up to e86542038dda88dadf8959584e803895f979310c #17047
   
   Closes #20605

   Changed files

   • .clang-tidy.yml
   • .github/workflows/linux.yml
   • CMake/Macros.cmake
   • CMakeLists.txt
   • Makefile.am
   • configure.ac
   • docs/INTERNALS.md
   • lib/Makefile.am
   • src/Makefile.am
   • src/tool_parsecfg.c

4. Change #258068

   Category	curl
   Changed by	Viktor Szakats <commitohnoyoudont@vsz.me>
   Changed at	Thu 19 Feb 2026 12:38:49
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	7e203c15e6be738944a467212a4a4d232af390f7

   Comments

   cmake/FindMbedTLS: add workaround for missing static MSVC `mbedcrypto.lib` 4.0.0
   Seen with mbedTLS 4.0.0. mbedTLS 4.0.0 renamed `mbedcrypto` lib to
   `tfpsacrypto`, while also keeping a copy under the old name to aid
   transition. However, this compatibility logic is broken for MSVC static
   builds, and the old name missing.
   
   Work around by looking for the new name in the raw detection codepath.
   
   Note that using `pkg-config`-based detection also works as a workaround.
   
   Reported-by: tawmoto on github
   Fixes #20616
   Ref: https://github.com/Mbed-TLS/mbedtls/blob/v4.0.0/library/CMakeLists.txt#L275-L282
   Ref: https://github.com/Mbed-TLS/mbedtls/issues/10605
   
   Closes #20617

   Changed files

   • CMake/FindMbedTLS.cmake

5. Change #258069

   Category	curl
   Changed by	Viktor Szakats <commitohnoyoudont@vsz.me>
   Changed at	Thu 19 Feb 2026 12:38:49
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	7c01bb23bc8454427149f31f4dae0c60c2991c15

   Comments

   rtspd: fix to check `realloc()` result
   Also enable `bugprone-suspicious-realloc-usage` clang-tidy option
   to verify.
   
   Fixing:
   ```
   tests/server/rtspd.c:328:37: error: 'req->rtp_buffer' may be set to null if 'realloc' fails,
    which may result in a leak of the original buffer
    [bugprone-suspicious-realloc-usage,-warnings-as-errors]
     328 |                   req->rtp_buffer = realloc(req->rtp_buffer,
         |                   ~~~~~~~~~~~~~~~   ^       ~~~~~~~~~~~~~~~
   ```
   
   Ref: https://clang.llvm.org/extra/clang-tidy/checks/bugprone/suspicious-realloc-usage.html
   
   Closes #20621

   Changed files

   • .clang-tidy.yml
   • tests/server/rtspd.c

6. Change #258071

   Category	curl
   Changed by	Viktor Szakats <commitohnoyoudont@vsz.me>
   Changed at	Thu 19 Feb 2026 12:44:37
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	c878160e9c1f7366e64299aa02540d5495c3df9c

   Comments

   clang-tidy: sync argument names in prototype and definition
   Discovered with clang-tidy checker
   `readability-inconsistent-declaration-parameter-name`.
   
   Also:
   - do not enforce the above because of inconsistencies still present
     between public API prototypes and definitions. (Also betwen man page
     protos, and man page examples, and other parts of the code, e.g.
     `easy` vs `curl` vs `d` vs `handle`) Perhaps subject for a future
     effort:
     https://github.com/curl/curl/actions/runs/22166472728/job/64094691653
   - enable and fix `readability-named-parameter` where missing.
   
   Refs:
   https://clang.llvm.org/extra/clang-tidy/checks/readability/inconsistent-declaration-parameter-name.html
   https://clang.llvm.org/extra/clang-tidy/checks/readability/named-parameter.html
   
   Closes #20624

   Changed files

   • .clang-tidy.yml
   • lib/altsvc.c
   • lib/altsvc.h
   • lib/asyn-ares.c
   • lib/asyn-thrdd.c
   • lib/asyn.h
   • lib/cf-h1-proxy.h
   • lib/cfilters.c
   • lib/cfilters.h
   • lib/conncache.h
   • lib/connect.h
   • lib/cookie.h
   • lib/cshutdn.h
   • lib/curl_addrinfo.c
   • lib/curl_hmac.h
   • lib/curl_md5.h
   • lib/curl_sha256.h
   • lib/curlx/strdup.h
   • lib/curlx/strparse.h
   • lib/curlx/warnless.h
   • lib/doh.h
   • lib/ftp.c
   • lib/ftplistparser.h
   • lib/hmac.c
   • lib/hostip.c
   • lib/hostip.h
   • lib/hsts.h
   • lib/http2.c
   • lib/http2.h
   • lib/idn.h
   • lib/llist.h
   • lib/mime.c
   • lib/multi.c
   • lib/multiif.h
   • lib/netrc.c
   • lib/netrc.h
   • lib/parsedate.c
   • lib/parsedate.h
   • lib/request.h
   • lib/select.h
   • lib/sendf.c
   • lib/sendf.h
   • lib/setopt.h
   • lib/sha256.c
   • lib/splay.h
   • lib/strcase.h
   • lib/strequal.c
   • lib/uint-table.h
   • lib/url.h
   • lib/vauth/cleartext.c
   • lib/vauth/vauth.h
   • lib/vssh/vssh.c
   • lib/vtls/hostcheck.h
   • lib/vtls/openssl.c
   • lib/vtls/openssl.h
   • lib/vtls/vtls.c
   • lib/vtls/vtls.h
   • lib/vtls/vtls_scache.c
   • lib/ws.c
   • lib/ws.h
   • src/tool_easysrc.h
   • src/tool_getpass.c
   • src/tool_paramhlp.h
   • tests/libtest/first.h

7. Change #258072

   Category	curl
   Changed by	Viktor Szakats <commitohnoyoudont@vsz.me>
   Changed at	Thu 19 Feb 2026 12:44:52
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	bd60df527c782d8c907d8e2ea8bdb172860da8b0

   Comments

   clang-tidy: check `readability-redundant-control-flow`
   Also fix fallouts.
   
   Ref: https://clang.llvm.org/extra/clang-tidy/checks/readability/redundant-control-flow.html
   
   Closes #20625

   Changed files

   • .clang-tidy.yml
   • docs/examples/synctime.c
   • tests/server/tftpd.c