diff options
author | Brian Smith <brian@briansmith.org> | 2016-02-07 22:36:04 +0300 |
---|---|---|
committer | David Benjamin <davidben@google.com> | 2016-02-12 01:07:56 +0300 |
commit | 5ba06897be4fb001113f6e8ea2398538dbbb88f2 (patch) | |
tree | f880255fcbb4de630e0827e02ae741d155b9b5c9 /crypto/bio | |
parent | 46a4d6d7051775589180e46b3974b2c94dc9f2d7 (diff) |
Don't cast |OPENSSL_malloc|/|OPENSSL_realloc| result.
C has implicit conversion of |void *| to other pointer types so these
casts are unnecessary. Clean them up to make the code easier to read
and to make it easier to find dangerous casts.
Change-Id: I26988a672e8ed4d69c75cfbb284413999b475464
Reviewed-on: https://boringssl-review.googlesource.com/7102
Reviewed-by: David Benjamin <davidben@google.com>
Diffstat (limited to 'crypto/bio')
-rw-r--r-- | crypto/bio/buffer.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/crypto/bio/buffer.c b/crypto/bio/buffer.c index 9d0cb3c0..15574510 100644 --- a/crypto/bio/buffer.c +++ b/crypto/bio/buffer.c @@ -100,7 +100,7 @@ static int buffer_new(BIO *bio) { if (ctx->ibuf == NULL) { goto err1; } - ctx->obuf = (char *)OPENSSL_malloc(DEFAULT_BUFFER_SIZE); + ctx->obuf = OPENSSL_malloc(DEFAULT_BUFFER_SIZE); if (ctx->obuf == NULL) { goto err2; } @@ -340,13 +340,13 @@ static long buffer_ctrl(BIO *b, int cmd, long num, void *ptr) { p1 = ctx->ibuf; p2 = ctx->obuf; if (ibs > DEFAULT_BUFFER_SIZE && ibs != ctx->ibuf_size) { - p1 = (char *)OPENSSL_malloc(ibs); + p1 = OPENSSL_malloc(ibs); if (p1 == NULL) { goto malloc_error; } } if (obs > DEFAULT_BUFFER_SIZE && obs != ctx->obuf_size) { - p2 = (char *)OPENSSL_malloc(obs); + p2 = OPENSSL_malloc(obs); if (p2 == NULL) { if (p1 != ctx->ibuf) { OPENSSL_free(p1); |