diff options
author | jkim <jkim@FreeBSD.org> | 2016-09-22 15:55:27 +0000 |
---|---|---|
committer | jkim <jkim@FreeBSD.org> | 2016-09-22 15:55:27 +0000 |
commit | 3599587ae793583e28ee9f173cd8fd9a871306f3 (patch) | |
tree | c5dc8fe4f348f5db5683bbe8d4f3413a7440b924 /crypto/openssl/crypto/mem_clr.c | |
parent | 73e9e3e293d8e68ffafc5d04f0cc74a824c84185 (diff) | |
download | FreeBSD-src-3599587ae793583e28ee9f173cd8fd9a871306f3.zip FreeBSD-src-3599587ae793583e28ee9f173cd8fd9a871306f3.tar.gz |
MFS: r306195
Merge OpenSSL 1.0.2i.
Approved by: re (gjb, implicit), so (delphij)
Diffstat (limited to 'crypto/openssl/crypto/mem_clr.c')
-rw-r--r-- | crypto/openssl/crypto/mem_clr.c | 24 |
1 files changed, 9 insertions, 15 deletions
diff --git a/crypto/openssl/crypto/mem_clr.c b/crypto/openssl/crypto/mem_clr.c index ab85344..579e9d1 100644 --- a/crypto/openssl/crypto/mem_clr.c +++ b/crypto/openssl/crypto/mem_clr.c @@ -60,22 +60,16 @@ #include <string.h> #include <openssl/crypto.h> -unsigned char cleanse_ctr = 0; +/* + * Pointer to memset is volatile so that compiler must de-reference + * the pointer and can't assume that it points to any function in + * particular (such as memset, which it then might further "optimize") + */ +typedef void *(*memset_t)(void *,int,size_t); + +static volatile memset_t memset_func = memset; void OPENSSL_cleanse(void *ptr, size_t len) { - unsigned char *p = ptr; - size_t loop = len, ctr = cleanse_ctr; - - if (ptr == NULL) - return; - - while (loop--) { - *(p++) = (unsigned char)ctr; - ctr += (17 + ((size_t)p & 0xF)); - } - p = memchr(ptr, (unsigned char)ctr, len); - if (p) - ctr += (63 + (size_t)p); - cleanse_ctr = (unsigned char)ctr; + memset_func(ptr, 0, len); } |