diff options
| author |   jsing <jsing@openbsd.org> | 2020-03-12 17:15:33 +0000 |
|---|---|---|
| committer | jsing <jsing@openbsd.org> | 2020-03-12 17:15:33 +0000 |
| commit | faf44e99eedfc4e5b95dbd5868fe860af04b7fef (patch) | |
| tree | 3e33674af293fbfb1fd8aae6d670e48f0897b2c4 /lib/libssl/ssl_both.c | |
| parent | Stop overloading the record type for padding length. (diff) | |
| download | wireguard-openbsd-faf44e99eedfc4e5b95dbd5868fe860af04b7fef.tar.xz wireguard-openbsd-faf44e99eedfc4e5b95dbd5868fe860af04b7fef.zip | |
Use calloc() rather than malloc() when allocating buffers.
This reduces the chance of accidently leaking stack memory.
ok inoguchi@ tb@
Diffstat (limited to 'lib/libssl/ssl_both.c')
| -rw-r--r-- | lib/libssl/ssl_both.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/libssl/ssl_both.c b/lib/libssl/ssl_both.c index 8ec94542c22..b8929d8f848 100644 --- a/lib/libssl/ssl_both.c +++ b/lib/libssl/ssl_both.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ssl_both.c,v 1.16 2020/01/23 10:48:37 jsing Exp $ */ +/* $OpenBSD: ssl_both.c,v 1.17 2020/03/12 17:15:33 jsing Exp $ */ /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) * All rights reserved. * @@ -702,7 +702,7 @@ ssl3_setup_read_buffer(SSL *s) if (S3I(s)->rbuf.buf == NULL) { len = SSL3_RT_MAX_PLAIN_LENGTH + SSL3_RT_MAX_ENCRYPTED_OVERHEAD + headerlen + align; - if ((p = malloc(len)) == NULL) + if ((p = calloc(1, len)) == NULL) goto err; S3I(s)->rbuf.buf = p; S3I(s)->rbuf.len = len; @@ -736,7 +736,7 @@ ssl3_setup_write_buffer(SSL *s) len += headerlen + align + SSL3_RT_SEND_MAX_ENCRYPTED_OVERHEAD; - if ((p = malloc(len)) == NULL) + if ((p = calloc(1, len)) == NULL) goto err; S3I(s)->wbuf.buf = p; S3I(s)->wbuf.len = len; |