diff options
author | Kamil Dudka <kdudka@redhat.com> | 2010-04-07 10:07:29 +0200 |
---|---|---|
committer | Kamil Dudka <kdudka@redhat.com> | 2010-04-07 10:47:19 +0200 |
commit | 10977f57de0d2426f479ef4d36f8a203488fdda7 (patch) | |
tree | 536506f3ba823c43931d681d3787aa6754704756 | |
parent | ef1ac363ee5ece033b32653b5c00a915b68ab8ad (diff) |
qssl: reflect recent code changes in SSL interface
Reported by Guenter Knauf.
-rw-r--r-- | lib/qssl.c | 23 | ||||
-rw-r--r-- | lib/qssl.h | 8 | ||||
-rw-r--r-- | lib/sendf.c | 2 |
3 files changed, 22 insertions, 11 deletions
diff --git a/lib/qssl.c b/lib/qssl.c index 3d1b5da20..5a08a2559 100644 --- a/lib/qssl.c +++ b/lib/qssl.c @@ -374,8 +374,9 @@ int Curl_qsossl_shutdown(struct connectdata * conn, int sockindex) } +/* for documentation see Curl_ssl_send() in sslgen.h */ ssize_t Curl_qsossl_send(struct connectdata * conn, int sockindex, - const void * mem, size_t len) + const void * mem, size_t len, int * curlcode) { /* SSL_Write() is said to return 'int' while write() and send() returns @@ -391,22 +392,26 @@ ssize_t Curl_qsossl_send(struct connectdata * conn, int sockindex, /* The operation did not complete; the same SSL I/O function should be called again later. This is basicly an EWOULDBLOCK equivalent. */ - return 0; + *curlcode = -1; /* EWOULDBLOCK */ + return -1; case SSL_ERROR_IO: switch (errno) { case EWOULDBLOCK: case EINTR: - return 0; + *curlcode = -1; /* EWOULDBLOCK */ + return -1; } failf(conn->data, "SSL_Write() I/O error: %s", strerror(errno)); + *curlcode = CURLE_SEND_ERROR; return -1; } /* An SSL error. */ failf(conn->data, "SSL_Write() returned error %s", SSL_Strerror(rc, NULL)); + *curlcode = CURLE_SEND_ERROR; return -1; } @@ -414,8 +419,9 @@ ssize_t Curl_qsossl_send(struct connectdata * conn, int sockindex, } +/* for documentation see Curl_ssl_recv() in sslgen.h */ ssize_t Curl_qsossl_recv(struct connectdata * conn, int num, char * buf, - size_t buffersize, bool * wouldblock) + size_t buffersize, int * curlcode) { char error_buffer[120]; /* OpenSSL documents that this must be at @@ -426,7 +432,6 @@ ssize_t Curl_qsossl_recv(struct connectdata * conn, int num, char * buf, buffsize = (buffersize > (size_t)INT_MAX) ? INT_MAX : (int)buffersize; nread = SSL_Read(conn->ssl[num].handle, buf, buffsize); - *wouldblock = FALSE; if(nread < 0) { /* failed SSL_read */ @@ -435,21 +440,23 @@ ssize_t Curl_qsossl_recv(struct connectdata * conn, int num, char * buf, case SSL_ERROR_BAD_STATE: /* there's data pending, re-invoke SSL_Read(). */ - *wouldblock = TRUE; - return -1; /* basically EWOULDBLOCK */ + *curlcode = -1; /* EWOULDBLOCK */ + return -1; case SSL_ERROR_IO: switch (errno) { case EWOULDBLOCK: - *wouldblock = TRUE; + *curlcode = -1; /* EWOULDBLOCK */ return -1; } failf(conn->data, "SSL_Read() I/O error: %s", strerror(errno)); + *curlcode = CURLE_RECV_ERROR; return -1; default: failf(conn->data, "SSL read error: %s", SSL_Strerror(nread, NULL)); + *curlcode = CURLE_RECV_ERROR; return -1; } } diff --git a/lib/qssl.h b/lib/qssl.h index 9a984bf4d..a0cf8cc29 100644 --- a/lib/qssl.h +++ b/lib/qssl.h @@ -36,15 +36,19 @@ void Curl_qsossl_close(struct connectdata *conn, int sockindex); int Curl_qsossl_close_all(struct SessionHandle * data); int Curl_qsossl_shutdown(struct connectdata * conn, int sockindex); +/* for documentation see Curl_ssl_send() in sslgen.h */ ssize_t Curl_qsossl_send(struct connectdata * conn, int sockindex, const void * mem, - size_t len); + size_t len, + int * curlcode); + +/* for documentation see Curl_ssl_recv() in sslgen.h */ ssize_t Curl_qsossl_recv(struct connectdata * conn, /* connection data */ int num, /* socketindex */ char * buf, /* store read data here */ size_t buffersize, /* max amount to read */ - bool * wouldblock); + int * curlcode); size_t Curl_qsossl_version(char * buffer, size_t size); int Curl_qsossl_check_cxn(struct connectdata * cxn); diff --git a/lib/sendf.c b/lib/sendf.c index 90c527558..ff4aee30b 100644 --- a/lib/sendf.c +++ b/lib/sendf.c @@ -553,7 +553,7 @@ int Curl_read(struct connectdata *conn, /* connection data */ } if(conn->ssl[num].state == ssl_connection_complete) { - int curlcode; + int curlcode = CURLE_RECV_ERROR; nread = Curl_ssl_recv(conn, num, buffertofill, bytesfromsocket, &curlcode); if(nread == -1) |