Skip to content

add a CUSTOM_SOCKET_ERROR_FUNCTION define for SET_SOCKERRNO#21

Closed
omork wants to merge 2 commits intocurl:masterfrom
agoragames:43037466987643237dc4e4d817c221f88c2adc35
Closed

add a CUSTOM_SOCKET_ERROR_FUNCTION define for SET_SOCKERRNO#21
omork wants to merge 2 commits intocurl:masterfrom
agoragames:43037466987643237dc4e4d817c221f88c2adc35

Conversation

@omork
Copy link

@omork omork commented May 26, 2011

Not sure why github won't separate these commits, but the second one is new.

The platform I'm developing on doesn't have winsock.h or a standard errno for socket errors.

I need to be able to define my own sockerrno/set_sockerrno. This patch allows me to set it outside of the curl headers and it won't be clobbered.

Ola Mork added 2 commits May 20, 2011 15:25
…ssl 1.9.0

Otherwise I'd get linker errors for the missing "SSL_CTX_load_verify_locations" functions:

--snip-- Debug/libcurl.lib(cyassl.obj): In function `cyassl_connect_step1':
--snip-- external/curl-7.21.6/lib/cyassl.c(127): undefined reference to `.SSL_CTX_load_verify_locations'
--snip-- external/curl-7.21.6/lib/cyassl.c(164): undefined reference to `.SSL_CTX_use_certificate_file'
--snip-- external/curl-7.21.6/lib/cyassl.c(172): undefined reference to `.SSL_CTX_use_PrivateKey_file'
@bagder
Copy link
Member

bagder commented May 31, 2011

Please send the patch(es) to the mailing list for review and discussion.

@bagder bagder closed this May 31, 2011
@bagder
Copy link
Member

bagder commented Aug 7, 2011

On Thu, 26 May 2011, omork wrote:

I need to be able to define my own sockerrno/set_sockerrno. This patch
allows me to set it outside of the curl headers and it won't be clobbered.

Can you please look at the current git repo after Yang Tse's reorg of defines
and read http://curl.haxx.se/mail/lib-2011-08/0027.html ?

And then, please take all follow-ups (and patch submissions) on the mailing
list and don't bother to post them on github.

/ daniel.haxx.se

@usergoodvery usergoodvery mentioned this pull request Nov 9, 2016
@lock lock bot locked as resolved and limited conversation to collaborators Jan 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants