Builder curl-threaded-solaris10-sparc Build #2812

Results:

Build successful

SourceStamp:

BuildSlave:

Reason:

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

Steps and Logfiles:

1. git update ( 2 secs )
   1. stdio
2. Runtest './tests/testcurl.pl --nogitpull ...' ( 3 hrs, 16 mins, 57 secs )
   1. stdio
   2. resultlog
3. Mail result 'cat resultlog ...' ( 19 secs )
   1. stdio

Build Properties:

Forced Build Properties:

Responsible Users:

1. Daniel Stenberg

   danielohnoyoudont@haxx.se
2. Stefan Eissing

   stefanohnoyoudont@eissing.org

Timing:

All Changes:

:

1. Change #230301

   Category	curl
   Changed by	Stefan Eissing <stefanohnoyoudont@eissing.org>
   Changed at	Tue 20 May 2025 14:49:17
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	be45e014c6ae82ae7752ca874ecefdc5bbc000fa

   Comments

   dns_entry: move from conn to data->state
   The `struct Curl_dns_entry *` used to established a connection
   do not have the connection's lifetime, but the transfer's lifetime
   (of the transfer that initiates the connect).
   
   `Curl_dns_entry *` is reference counted with the "dns cache". That
   cache might be owned by the multi or the transfer's share. In the
   share, the reference count needs updating under lock.
   
   Therefore, the dns entry can only be kept *and* released using the
   same transfer it was initially looked up from. But a connection is
   often discarded using another transfer.
   
   So far, the problem of this has been avoided in clearing the connection's
   dns entries in the "multi_don()" handling. So, connections had NULL
   dns entries after the initial transfers and its connect had been handled.
   
   Keeping the dns entries in data->state seems therefore a better choice.
   
   Also: remove the `struct Curl_dns_entry *` from the connect filters
   contexts. Use `data->state.dns` every time instead and fail correctly
   when not present and needed.
   
   Closes #17383

   Changed files

   • lib/cf-https-connect.c
   • lib/cf-https-connect.h
   • lib/connect.c
   • lib/connect.h
   • lib/cshutdn.c
   • lib/easy.c
   • lib/ftp.c
   • lib/hostip.c
   • lib/hostip.h
   • lib/multi.c
   • lib/setopt.c
   • lib/url.c
   • lib/url.h
   • lib/urldata.h

2. Change #230310

   Category	curl
   Changed by	Daniel Stenberg <danielohnoyoudont@haxx.se>
   Changed at	Tue 20 May 2025 16:08:29
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	155af7ac777e3f39e7122b9d14cc031357316564

   Comments

   ftplistparser: add two overflow preventions
   These two conditions probably cannot actually happen, but these two
   checks make that certain and should please the static code analyzers.
   
   Pointed out by Coverity
   
   Closes #17397

   Changed files

   • lib/ftplistparser.c