Skip to content

GHA/windows: add MSVC wolfSSL job with test #14193

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 4 commits into from

Conversation

talregev
Copy link
Contributor

@talregev talregev commented Jul 15, 2024

Fix the file of wolfssl.c because of this warning/error:

curl\lib\vtls\wolfssl.c(1017,42): error C2220: the following warning is treated as an error [curl\bld\lib\libcurl_object.vcxproj]
curl\lib\vtls\wolfssl.c(1017,42): warning C4267: 'function': conversion from 'size_t' to 'unsigned long', possible loss of data [curl\bld\lib\libcurl_object.vcxproj]

size_t in MSVC is different. Change it to unsigned long because
wolfSSL_ERR_error_string_n last argument is defined as
unsigned long.

Closes #14193

@github-actions github-actions bot added the CI Continuous Integration label Jul 15, 2024
@vszakats vszakats added the Windows Windows-specific label Jul 15, 2024
@vszakats vszakats changed the title GHA/windows: Add wolfssl lib to ci + test GHA/windows: add msvc wolfssl job with test Jul 15, 2024
@vszakats vszakats changed the title GHA/windows: add msvc wolfssl job with test GHA/windows: add MSVC wolfSSL job with test Jul 16, 2024
@vszakats vszakats closed this in dfbdcb9 Jul 16, 2024
@vszakats
Copy link
Member

Thanks @talregev, merged!

meslubi2021 pushed a commit to Unity-Curl/curl that referenced this pull request Jul 19, 2024
Fix the file of wolfssl.c because of this warning/error:
```
curl\lib\vtls\wolfssl.c(1017,42): error C2220: the following warning is treated as an error [curl\bld\lib\libcurl_object.vcxproj]
curl\lib\vtls\wolfssl.c(1017,42): warning C4267: 'function': conversion from 'size_t' to 'unsigned long', possible loss of data [curl\bld\lib\libcurl_object.vcxproj]
```

`size_t` in MSVC is different. Change it to `unsigned long` because
`wolfSSL_ERR_error_string_n` last argument is defined as
`unsigned long`.

Closes curl#14193
@talregev talregev deleted the TalR/wolfssl branch July 23, 2024 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration Windows Windows-specific
Development

Successfully merging this pull request may close these issues.

2 participants