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

Builder curl-pr-unthreaded-solaris10-i386 Build #3573

Results:

Build successful

SourceStamp:

Projectcurl/curl
Repositoryhttps://github.com/curl/curl
Branchmaster
Revisionb06c12b7248592cf001e621d7cd8dc78a827212b
Got Revisionb06c12b7248592cf001e621d7cd8dc78a827212b
Changes16 changes

BuildSlave:

unstable10x

Reason:

The AnyBranchScheduler scheduler named 'schedule-curl-pr' triggered this build

Steps and Logfiles:

  1. git update ( 8 secs )
    1. stdio
  2. Runtest './tests/testcurl.pl --nogitpull ...' ( 1 hrs, 19 mins, 25 secs )
    1. stdio
    2. resultlog

Build Properties:

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

Forced Build Properties:

NameLabelValue

Responsible Users:

  1. Daniel Stenberg
  2. Ondřej Hlavatý
  3. Orgad Shaneh
  4. Stefan Eissing
  5. Viktor Szakats

Timing:

StartWed Mar 5 17:47:40 2025
EndWed Mar 5 23:22:13 2025
Elapsed5 hrs, 34 mins, 33 secs

All Changes:

:

  1. Change #223861

    Category None
    Changed by Daniel Stenberg <danielohnoyoudont@haxx.se>
    Changed at Wed 05 Mar 2025 07:44:43
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision d121bc9884312fe4d92a94b545dc9b93259dd1cc

    Comments

    tool_getparam: parse --trace-config without strdup()/free()
    
    Closes #16559

    Changed files

    • src/tool_getparam.c
  2. Change #223868

    Category None
    Changed by Orgad Shaneh <orgad.shanehohnoyoudont@audiocodes.com>
    Changed at Wed 05 Mar 2025 08:19:15
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision 7298c4320bcc16cfa21c26fecb7568c7c42d785d

    Comments

    docs: adapt to removed --with-random
    
    Amends 269fdd4c6ed5d837d57448ac977f6f300968df15.
    
    Closes #16565

    Changed files

    • docs/INSTALL.md
  3. Change #223872

    Category None
    Changed by Daniel Stenberg <danielohnoyoudont@haxx.se>
    Changed at Wed 05 Mar 2025 09:19:09
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision ed15fce1fddd5f0e3dd1526fb50b803953754a17

    Comments

    src: avoid strdup on platforms not doing UTF8 conversions
    
    ... and use more const strings.
    
    Closes #16560

    Changed files

    • lib/curl_multibyte.h
    • src/tool_getparam.c
    • src/tool_getparam.h
    • src/tool_operate.c
    • src/tool_stderr.c
    • src/tool_stderr.h
  4. Change #223890

    Category None
    Changed by Stefan Eissing <stefanohnoyoudont@eissing.org>
    Changed at Wed 05 Mar 2025 10:58:52
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision e7de83aa2982a9be84a0f73a2533742702bd6351

    Comments

    http2: reset stream on response header error
    
    We send a GOAWAY, but some servers ignore that and happily continue
    sending the stream response. RST the stream when response header errors
    are encountered.
    
    Fixes #16535
    Reported-by: Peng-Yu Chen
    Closes #16544

    Changed files

    • lib/http2.c
  5. Change #223891

    Category None
    Changed by Stefan Eissing <stefanohnoyoudont@eissing.org>
    Changed at Wed 05 Mar 2025 10:59:19
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision aa042210d9033d3009681991cdaf6feaca69904d

    Comments

    http2: add on_invalid_frame callback for error detection
    
    When the server sends HEADER/CONTINUATION frames that exceed nghttp2's
    size, this error is being reported via the on_invalid_frame_recv
    callback. Without registering there, it will go unnoticed.
    
    RST the stream when such a frame is encountered.
    
    Closes #16544

    Changed files

    • lib/http2.c
    • tests/http/test_07_upload.py
  6. Change #223892

    Category None
    Changed by Stefan Eissing <stefanohnoyoudont@eissing.org>
    Changed at Wed 05 Mar 2025 10:59:27
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision e542fd9da10634bf11b1b6185a380712909dbc10

    Comments

    http2: detect session being closed on ingress handling
    
    nghttp2 will on its own send GOAWAY frames, closing the connection, when
    internal processing of frames runs into errors. This may not become
    visible in a direct error code from a call to nghttp2.
    
    Check for session being closed on ingress processing (on sending, we
    already did that) and report an error if so. In addition, monitor
    outgoing GOAWAY not initiated by us so that the user will get a fail
    message when that happens.
    
    Add some more long response header tests.
    
    Closes #16544

    Changed files

    • lib/http2.c
    • tests/http/test_01_basic.py
  7. Change #223902

    Category None
    Changed by Daniel Stenberg <danielohnoyoudont@haxx.se>
    Changed at Wed 05 Mar 2025 11:37:38
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision 34cbf95267f7c3f7b95e4c6c011dac79d6925993

    Comments

    tests: make sure 'commands.log' is generated in the correct logdir
    
    Closes #16568

    Changed files

    • tests/runner.pm
  8. Change #223908

    Category None
    Changed by Daniel Stenberg <danielohnoyoudont@haxx.se>
    Changed at Wed 05 Mar 2025 11:50:06
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision c4cd0ae9ba3fb28ff316202d15b5a5b4a23a116a

    Comments

    http_aws_sigv4: cannot be used for proxy
    
    Make sure it is never attempted.
    
    Reported-by: Philippe Antoine
    Closes #16569

    Changed files

    • lib/http.c
    • lib/http_aws_sigv4.c
    • lib/http_aws_sigv4.h
  9. Change #223921

    Category None
    Changed by Daniel Stenberg <danielohnoyoudont@haxx.se>
    Changed at Wed 05 Mar 2025 13:24:45
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision 86ac471dc62893f47bce3cb0aa6c797588056384

    Comments

    tool_paramhlp: do --proto parsing without strtok
    
    Saves a strdup. Avoids a banned function.
    
    Closes #16567

    Changed files

    • src/tool_paramhlp.c
  10. Change #223922

    Category None
    Changed by Daniel Stenberg <danielohnoyoudont@haxx.se>
    Changed at Wed 05 Mar 2025 13:26:33
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision 787ee935acd5867bdac836b2043b6095eed2c29e

    Comments

    tool_dirhie: create dir hierarchy without strtok
    
    And use dynbuf
    
    Closes #16566

    Changed files

    • src/tool_dirhie.c
  11. Change #223925

    Category None
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Wed 05 Mar 2025 13:34:16
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision e6c78e18dac1da2027eac6dd3829a0fdbfa55501

    Comments

    GHA/windows: always install diffutils for MSYS/mingw, required by `runtests`
    
    To include the expected/generated diffs in the error log.
    
    Also make it explicit for pure MSYS, though it was installed by `gcc`
    before this patch.
    
    Closes #16571

    Changed files

    • .github/workflows/windows.yml
  12. Change #223933

    Category None
    Changed by Stefan Eissing <stefanohnoyoudont@eissing.org>
    Changed at Wed 05 Mar 2025 13:51:56
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision 9bfa64f850ec03a12385d9a5d5557dffcec267cc

    Comments

    gnutls: set priority via --ciphers
    
    No longer ignore the `--ciphers` argument in gnutls curl builds, but use
    it to set the gnutls priority string.
    
    When the set ciphers start with '+', '-' or '!', it is *appended* to the
    curl generated priority string. Otherwise it replaces the curl one
    completely.
    
    Add test_17_18 to check various combinations.
    
    Closes #16557

    Changed files

    • .github/scripts/spellcheck.words
    • docs/CIPHERS.md
    • docs/libcurl/opts/CURLOPT_SSL_CIPHER_LIST.md
    • lib/vtls/gtls.c
    • tests/http/test_17_ssl_use.py
  13. Change #223939

    Category None
    Changed by Ondřej Hlavatý <aearsisohnoyoudont@eideo.cz>
    Changed at Wed 05 Mar 2025 14:54:33
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision 5a3fe980c56f5af37698cd30866259daf8fb6589

    Comments

    request: clear sendbuf_hds_len when resetting request bufq
    
    Without this, any usage of sendbuf_hds_len on a retried request is
    wrong. We noticed by getting debug callbacks with incorrect header len.
    We did not figure out how to trigger the retries in a test environment
    though.
    
    Closes #16573

    Changed files

    • lib/request.c
  14. Change #223940

    Category None
    Changed by Daniel Stenberg <danielohnoyoudont@haxx.se>
    Changed at Wed 05 Mar 2025 14:55:55
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision f79195e7efea75c3109abac17d8637ed3592083f

    Comments

    tool_getparam: parse_header() without strtok
    
    Read a provided header file line-by-line instead. Avoids strtok(),
    avoids a possibly quite large malloc for the entire file.
    
    Closes #16572

    Changed files

    • src/tool_getparam.c
  15. Change #223952

    Category None
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Wed 05 Mar 2025 17:26:11
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision 1bd5ac998bbc943dbf812b2824ad0f532201734c

    Comments

    GHA/windows: stop ignoring most ignored test results
    
    These tests seem to be running no less stable now than others.
    Stop ignoring their results to catch real issues.
    
    These are consistently failing and remain on the ignore list:
    
    in MSVC / vcpkg jobs:
    ```
    FAIL-IGNORED 2302: 'WebSockets via callback (frame mode) + curl_ws_send()' WebSockets
    FAIL-IGNORED 2303: 'WebSockets but gets a 200 back' WebSockets
    FAIL-IGNORED 2307: 'WebSockets, overlong PING payload' WebSockets
    ```
    https://github.com/curl/curl/actions/runs/13674664461/job/38233949942?pr=16570#step:14:4089
    - Likely curl issues either in tests, server, or in WebSockets support.
    
    in tests running under MSYS, affecting native mingw Windows builds only:
    ```
    FAIL-IGNORED 612: 'SFTP post-quote remove file' SFTP, post-quote
    [...]
     curl: (21) rm command failed: Operation failed
    ```
    https://github.com/curl/curl/actions/runs/13674664461/job/38233952699?pr=16570#step:14:1378
    
    in tests running under MSYS, affecting both MSYS and native mingw Windows builds:
    ```diff
    FAIL-IGNORED 613: 'SFTP directory retrieval' SFTP, directory
    [...]
     --- log/7/check-expected	2025-03-05 11:19:54.119658000 +0000
     +++ log/7/check-generated	2025-03-05 11:19:54.119658000 +0000
    @@ -1,3 +1,3 @@
     d?????????    N U         U               N ???  N NN:NN asubdir[LF]
    --rw?rw?rw?    1 U         U              37 Jan  1  2000 plainfile.txt[LF]
    +-rw?r-?r-?    1 U         U              37 Jan  1  2000 plainfile.txt[LF]
     -r-?r-?r-?    1 U         U              47 Dec 31  2000 rofile.txt[LF]
    ```
    https://github.com/curl/curl/actions/runs/13674664461/job/38233950866?pr=16570#step:14:1316
    - Possibly a curl test portabibility, Perl or MSYS issue.
    
    in Cygwin tests:
    ```
    FAIL-IGNORED 615: 'SFTP put remote failure' SFTP, SFTP put, FAILURE
    ```
    https://github.com/curl/curl/actions/runs/13674664461/job/38233949428?pr=16570#step:12:3817
    
    Follow-up to adcfd4fb3e9be1de0e506728066bea2aaa53c394 #16553
    Ref: #14854
    Closes #16570

    Changed files

    • .github/workflows/windows.yml
  16. Change #223953

    Category None
    Changed by Viktor Szakats <commitohnoyoudont@vsz.me>
    Changed at Wed 05 Mar 2025 17:26:12
    Repository https://github.com/curl/curl
    Project curl/curl
    Branch master
    Revision b06c12b7248592cf001e621d7cd8dc78a827212b

    Comments

    GHA/windows: bump msys2 action, downgrade runtime for mingw tests
    
    Bump msys2/setup-msys2 from 2.26.0 to 2.27.0. It brings the perf
    regression experienced earlier with GfW and the pre-installed
    MSYS2 on the GHA runner. Apply the runtime downgrade trick as
    a workaround.
    
    Fixes #16440
    Fixes #16547
    Closes #16574

    Changed files

    • .github/workflows/windows.yml