Merge pull request #8572

50b7492 unit_tests: suppress memwipe unit warning (Jeffrey Ryan)
This commit is contained in:
luigi1111 2022-12-01 01:01:34 -06:00
commit 12489c69cb
No known key found for this signature in database
GPG key ID: F4ACA0183641E010

View file

@ -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);
}