Fix segfault in allocator_tests/arena_tests

jkczyz · furszy · commit 929556868753 · 2021-08-04T14:11:04.000-03:00
The test uses reinterpret_cast&lt;void*&gt; on unallocated memory. Using this
memory in printchunk as char* causes a segfault, so have printchunk take
void* instead.
diff --git a/src/support/lockedpool.cpp b/src/support/lockedpool.cpp
@@ -145,7 +145,7 @@ Arena::Stats Arena::stats() const
 }
 
 #ifdef ARENA_DEBUG
-void printchunk(char* base, size_t sz, bool used) {
+static void printchunk(void* base, size_t sz, bool used) {
     std::cout <<
         "0x" << std::hex << std::setw(16) << std::setfill('0') << base <<
         " 0x" << std::hex << std::setw(16) << std::setfill('0') << sz <<

Original file line number	Diff line number	Diff line change
`@@ -145,7 +145,7 @@ Arena::Stats Arena::stats() const`
`145`	`145`	`}`
`146`	`146`
`147`	`147`	`#ifdef ARENA_DEBUG`
`148`		`-void printchunk(char* base, size_t sz, bool used) {`
	`148`	`+static void printchunk(void* base, size_t sz, bool used) {`
`149`	`149`	`std::cout <<`
`150`	`150`	`"0x" << std::hex << std::setw(16) << std::setfill('0') << base <<`
`151`	`151`	`" 0x" << std::hex << std::setw(16) << std::setfill('0') << sz <<`