diff options
| author | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2018-07-10 14:37:56 +0100 |
|---|---|---|
| committer | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2018-07-10 15:37:22 +0100 |
| commit | 8d578f1f2df872be93d6fd2f3c4a4aef40d9faa3 (patch) | |
| tree | 525bea5ece9a76730d7d8766e36f11086d1fedc6 | |
| parent | Merge pull request #4094 (diff) | |
| download | monero-8d578f1f2df872be93d6fd2f3c4a4aef40d9faa3.tar.xz | |
memwipe: don't call the workhorse for 0 bytes
Some of them don't like it
| -rw-r--r-- | contrib/epee/src/memwipe.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/contrib/epee/src/memwipe.c b/contrib/epee/src/memwipe.c index e3a2f76c8..c2a26c392 100644 --- a/contrib/epee/src/memwipe.c +++ b/contrib/epee/src/memwipe.c @@ -50,7 +50,7 @@ void *memwipe(void *ptr, size_t n) { - if (memset_s(ptr, n, 0, n)) + if (n > 0 && memset_s(ptr, n, 0, n)) { #ifdef NDEBUG fprintf(stderr, "Error: memset_s failed\n"); @@ -67,7 +67,8 @@ void *memwipe(void *ptr, size_t n) void *memwipe(void *ptr, size_t n) { - explicit_bzero(ptr, n); + if (n > 0) + explicit_bzero(ptr, n); SCARECROW return ptr; } @@ -105,7 +106,8 @@ static void memory_cleanse(void *ptr, size_t len) void *memwipe(void *ptr, size_t n) { - memory_cleanse(ptr, n); + if (n > 0) + memory_cleanse(ptr, n); SCARECROW return ptr; } |