diff options
author | Jeffrey Ryan <jeffreyryan@tutanota.com> | 2022-09-16 10:45:32 -0500 |
---|---|---|
committer | Jeffrey Ryan <jeffreyryan@tutanota.com> | 2022-09-16 10:47:00 -0500 |
commit | 50b7492e67b314192012f7a776c635999317bfb9 (patch) | |
tree | 8ea0b0fca3e3373b54e8d42206a8d6ab2e9aaf6c | |
parent | Merge pull request #8554 (diff) | |
download | monero-50b7492e67b314192012f7a776c635999317bfb9.tar.xz |
unit_tests: suppress memwipe unit warning
Fixes warning:
```
warning: ‘*(uint32_t*)quux’ may be used uninitialized [-Wmaybe-uninitialized]
49 | MDEBUG(std::hex << std::setw(8) << std::setfill('0') << *(uint32_t*)quux);
warning: ‘quux’ may be used uninitialized [-Wmaybe-uninitialized]
50 | if (wipe) { ASSERT_TRUE(memcmp(quux, "bar", 3)); }
```
-rw-r--r-- | tests/unit_tests/memwipe.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/tests/unit_tests/memwipe.cpp b/tests/unit_tests/memwipe.cpp index 97f003471..8eab69548 100644 --- a/tests/unit_tests/memwipe.cpp +++ b/tests/unit_tests/memwipe.cpp @@ -31,6 +31,7 @@ #include <stdint.h> #include "misc_log_ex.h" #include "memwipe.h" +#include "warnings.h" // Probably won't catch the optimized out case, but at least we test // it works in the normal case @@ -44,12 +45,15 @@ static void test(bool wipe) ASSERT_EQ(foo, bar); free(foo); char *quux = (char*)malloc(4); // same size, just after free, so we're likely to get the same, depending on the allocator +PUSH_WARNINGS +DISABLE_GCC_WARNING(maybe-uninitialized) if ((intptr_t)quux == foop) { MDEBUG(std::hex << std::setw(8) << std::setfill('0') << *(uint32_t*)quux); if (wipe) { ASSERT_TRUE(memcmp(quux, "bar", 3)); } } else MWARNING("We did not get the same location, cannot check"); +POP_WARNINGS free(quux); } |