Skip to content

wolfssl: fix build for wolfssl without bio chain support - #22255

Closed
jay wants to merge 1 commit into
curl:masterfrom
jay:wolfssl_build_err
Closed

wolfssl: fix build for wolfssl without bio chain support#22255
jay wants to merge 1 commit into
curl:masterfrom
jay:wolfssl_build_err

Conversation

@jay

@jay jay commented Jul 3, 2026

Copy link
Copy Markdown
Member
  • Do not mix declarations and code. (ISO C90 build error)

Ref: #22252 (comment)

Closes #xxxx


../../lib/vtls/wolfssl.c: In function 'wssl_connect_step1':
../../lib/vtls/wolfssl.c:1542:3: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement]
 1542 |   curl_socket_t sockfd = Curl_conn_cf_get_socket(cf, data);
      |   ^~~~~~~~~~~~~

- Do not mix declarations and code. (ISO C90 build error)

Ref: curl#22252 (comment)

Closes #xxxx
@jay
jay requested a review from vszakats July 3, 2026 14:41
@jay jay added the build label Jul 3, 2026
@github-actions github-actions Bot added the TLS label Jul 3, 2026
@testclutch

Copy link
Copy Markdown

Analysis of PR #22255 at 60f9534f:

Test 593 failed, which has NOT been flaky recently, so there could be a real issue in this PR. Note that this test has failed in 10 different CI jobs (the link just goes to one of them).

Test 311 failed, but it has been 0.6% flaky lately, so it's probably NOT a fault of the PR. Note that this test has failed in 12 different CI jobs (the link just goes to one of them).

Generated by Testclutch

@jay jay closed this in 3dd6a87 Jul 3, 2026
@jay
jay deleted the wolfssl_build_err branch July 3, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants