Skip to content
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

Jump to bottom

libCurl build fails with CURL_DISABLE_CRYPTO_AUTH and USE_SSL (causes vtls.c to fail to compile) #1945

Closed
wyattoday opened this issue Oct 4, 2017 · 4 comments
Closed

libCurl build fails with CURL_DISABLE_CRYPTO_AUTH and USE_SSL (causes vtls.c to fail to compile) #1945

wyattoday opened this issue Oct 4, 2017 · 4 comments
Labels
build

Comments

@wyattoday
Copy link
Contributor

I did this

Defined both CURL_DISABLE_CRYPTO_AUTH and USE_SSL.

I expected the following

A successful compilation. Instead I get the following errors:

1>vtls\vtls.c(1065): error C2065: 'MD5_context' : undeclared identifier
1>vtls\vtls.c(1065): error C2065: 'MD5pw' : undeclared identifier
1>vtls\vtls.c(1069): error C2065: 'MD5pw' : undeclared identifier
1>vtls\vtls.c(1069): error C2065: 'Curl_DIGEST_MD5' : undeclared identifier
1>vtls\vtls.c(1070): error C2065: 'MD5pw' : undeclared identifier
1>vtls\vtls.c(1072): error C2065: 'MD5pw' : undeclared identifier
1>vtls\vtls.c(1073): error C2065: 'MD5pw' : undeclared identifier

curl/libcurl version

7.56.0

operating system

Windows, Visual Studio 2017.

A solution (but maybe not the best solution)

In vtls.c add #ifndef CURL_DISABLE_CRYPTO_AUTH around the Curl_none_md5sum function. Then set Curl_ssl_multi md5sum parameter to NULL (instead of Curl_none_md5sum). Do the same for Curl_ssl_schannel md5sum parameter.
@toonetown
Copy link

I am also seeing this when compiling for android, iOS, or macOS.

@bagder bagder added the build label Oct 4, 2017
bagder added a commit that referenced this issue Oct 4, 2017
@bagder
build: fix --disable-crypto-auth
a62f24e 
Reported-by: Wyatt O'Day
Fixes #1945
@bagder bagder mentioned this issue Oct 4, 2017
Closed
@bagder
Copy link
Member

bagder commented Oct 5, 2017

Let me know if (any of) you have any feedback or comments on my proposed fix. If not, I'll merge it soonish.

@wyattoday
Copy link
Contributor Author

wyattoday commented Oct 5, 2017 •
edited

Everything works now with that commit, thanks.

bagder added a commit that referenced this issue Oct 5, 2017
@bagder
build: fix --disable-crypto-auth
7bc5308 
Reported-by: Wyatt O'Day
Fixes #1945
Closes #1947
@bagder
Copy link
Member

bagder commented Oct 5, 2017

Thanks!

toonetown pushed a commit to toonetown/curl that referenced this issue Oct 5, 2017
@bagder @toonetown
build: fix --disable-crypto-auth
f2699a1 
Reported-by: Wyatt O'Day
Fixes curl#1945
Closes curl#1947
@bagder bagder mentioned this issue Oct 8, 2017
Closed
@lock lock bot locked as resolved and limited conversation to collaborators May 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
build
Milestone
No milestone
Development

No branches or pull requests
3 participants
@toonetown @bagder @wyattoday