diff options
| author | Matt Caswell <matt@openssl.org> | 2016-10-21 14:35:26 +0100 |
|---|---|---|
| committer | Matt Caswell <matt@openssl.org> | 2016-10-28 09:48:54 +0100 |
| commit | f42fd819d60c5ebbcfd7bff6173b89664ab2fde1 (patch) | |
| tree | 2045a9b8dbb35d826c298bccb5ca1cbacc640a41 /ssl/bio_ssl.c | |
| parent | bb5310bed5ab14747cad1f6a57aa3b075ca4af65 (diff) | |
Tweaks based on review feedback of BIO size_t work
Rename some parameters.
Also change handling of buffer sizes >INT_MAX in length.
Reviewed-by: Richard Levitte <levitte@openssl.org>
Diffstat (limited to 'ssl/bio_ssl.c')
| -rw-r--r-- | ssl/bio_ssl.c | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/ssl/bio_ssl.c b/ssl/bio_ssl.c index af5f913d8f..d64451c704 100644 --- a/ssl/bio_ssl.c +++ b/ssl/bio_ssl.c @@ -16,8 +16,8 @@ #include <openssl/err.h> #include "ssl_locl.h" -static int ssl_write(BIO *h, const char *buf, size_t num, size_t *written); -static int ssl_read(BIO *b, char *out, size_t outl, size_t *read); +static int ssl_write(BIO *h, const char *buf, size_t size, size_t *written); +static int ssl_read(BIO *b, char *buf, size_t size, size_t *readbytes); static int ssl_puts(BIO *h, const char *str); static long ssl_ctrl(BIO *h, int cmd, long arg1, void *arg2); static int ssl_new(BIO *h); @@ -88,7 +88,7 @@ static int ssl_free(BIO *a) return 1; } -static int ssl_read(BIO *b, char *out, size_t outl, size_t *readbytes) +static int ssl_read(BIO *b, char *buf, size_t size, size_t *readbytes) { int ret = 1; BIO_SSL *sb; @@ -96,17 +96,17 @@ static int ssl_read(BIO *b, char *out, size_t outl, size_t *readbytes) int retry_reason = 0; int r = 0; - if (out == NULL) - return (0); + if (buf == NULL) + return 0; sb = BIO_get_data(b); ssl = sb->ssl; BIO_clear_retry_flags(b); - if (outl > INT_MAX) - return -1; + if (size > INT_MAX) + size = INT_MAX; - ret = SSL_read(ssl, out, outl); + ret = SSL_read(ssl, buf, size); if (ret > 0) *readbytes = ret; @@ -165,24 +165,24 @@ static int ssl_read(BIO *b, char *out, size_t outl, size_t *readbytes) return ret; } -static int ssl_write(BIO *b, const char *out, size_t outl, size_t *written) +static int ssl_write(BIO *b, const char *buf, size_t size, size_t *written) { int ret, r = 0; int retry_reason = 0; SSL *ssl; BIO_SSL *bs; - if (out == NULL) - return (0); + if (buf == NULL) + return 0; bs = BIO_get_data(b); ssl = bs->ssl; BIO_clear_retry_flags(b); - if (outl > INT_MAX) - return 0; + if (size > INT_MAX) + size = INT_MAX; - ret = SSL_write(ssl, out, outl); + ret = SSL_write(ssl, buf, size); switch (SSL_get_error(ssl, ret)) { case SSL_ERROR_NONE: |