RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-10 09:37:34 +00:00
Code Issues Activity Actions

UserspaceEmulator: Use a report() function instead of dbgprintf()

Browse source 
Ultimately we'll want to make it a bit easier to add more reporting.
This at least makes it easier to redirect the logging.
This commit is contained in:
Andreas Kling 2020-07-31 18:54:30 +02:00
parent 628b3badfb
commit c7e4c0734b
4 changed files with 40 additions and 30 deletions
Download patch file Download diff file Expand all files Collapse all files

42
DevTools/UserspaceEmulator/MallocTracer.cpp
View file

@ -66,9 +66,9 @@ void MallocTracer::target_did_free(Badge<SoftCPU>, FlatPtr address)
for (auto& mallocation : m_mallocations) {
if (mallocation.address == address) {
if (mallocation.freed) {
dbgprintf("\n");
dbgprintf("==%d== \033[31;1mDouble free()\033[0m, %p\n", getpid(), address);
dbgprintf("==%d== Address %p has already been passed to free()\n", getpid(), address);
report("\n");
report("==%d== \033[31;1mDouble free()\033[0m, %p\n", getpid(), address);
report("==%d== Address %p has already been passed to free()\n", getpid(), address);
Emulator::the().dump_backtrace();
} else {
mallocation.freed = true;
@ -77,9 +77,9 @@ void MallocTracer::target_did_free(Badge<SoftCPU>, FlatPtr address)
return;
}
}
dbgprintf("\n");
dbgprintf("==%d== \033[31;1mInvalid free()\033[0m, %p\n", getpid(), address);
dbgprintf("==%d== Address %p has never been returned by malloc()\n", getpid(), address);
report("\n");
report("==%d== \033[31;1mInvalid free()\033[0m, %p\n", getpid(), address);
report("==%d== Address %p has never been returned by malloc()\n", getpid(), address);
Emulator::the().dump_backtrace();
}
@ -107,12 +107,12 @@ void MallocTracer::audit_read(FlatPtr address, size_t size)
size_t offset_into_mallocation = address - mallocation->address;
if (mallocation->freed) {
dbgprintf("\n");
dbgprintf("==%d== \033[31;1mUse-after-free\033[0m, invalid %zu-byte read at address %p\n", getpid(), size, address);
report("\n");
report("==%d== \033[31;1mUse-after-free\033[0m, invalid %zu-byte read at address %p\n", getpid(), size, address);
Emulator::the().dump_backtrace();
dbgprintf("==%d== Address is %zu bytes into block of size %zu, allocated at:\n", getpid(), offset_into_mallocation, mallocation->size);
report("==%d== Address is %zu bytes into block of size %zu, allocated at:\n", getpid(), offset_into_mallocation, mallocation->size);
Emulator::the().dump_backtrace(mallocation->malloc_backtrace);
dbgprintf("==%d== Later freed at:\n", getpid(), offset_into_mallocation, mallocation->size);
report("==%d== Later freed at:\n", getpid(), offset_into_mallocation, mallocation->size);
Emulator::the().dump_backtrace(mallocation->free_backtrace);
return;
}
@ -133,12 +133,12 @@ void MallocTracer::audit_write(FlatPtr address, size_t size)
size_t offset_into_mallocation = address - mallocation->address;
if (mallocation->freed) {
dbgprintf("\n");
dbgprintf("==%d== \033[31;1mUse-after-free\033[0m, invalid %zu-byte write at address %p\n", getpid(), size, address);
report("\n");
report("==%d== \033[31;1mUse-after-free\033[0m, invalid %zu-byte write at address %p\n", getpid(), size, address);
Emulator::the().dump_backtrace();
dbgprintf("==%d== Address is %zu bytes into block of size %zu, allocated at:\n", getpid(), offset_into_mallocation, mallocation->size);
report("==%d== Address is %zu bytes into block of size %zu, allocated at:\n", getpid(), offset_into_mallocation, mallocation->size);
Emulator::the().dump_backtrace(mallocation->malloc_backtrace);
dbgprintf("==%d== Later freed at:\n", getpid(), offset_into_mallocation, mallocation->size);
report("==%d== Later freed at:\n", getpid(), offset_into_mallocation, mallocation->size);
Emulator::the().dump_backtrace(mallocation->free_backtrace);
return;
}
@ -159,7 +159,7 @@ bool MallocTracer::is_reachable(const Mallocation& mallocation) const
auto value = Emulator::the().mmu().read32({ 0x20, other_mallocation.address + i * sizeof(u32) });
if (value.value() == mallocation.address && !value.is_uninitialized()) {
#ifdef REACHABLE_DEBUG
dbgprintf("mallocation %p is reachable from other mallocation %p\n", mallocation.address, other_mallocation.address);
report("mallocation %p is reachable from other mallocation %p\n", mallocation.address, other_mallocation.address);
#endif
return true;
}
@ -184,7 +184,7 @@ bool MallocTracer::is_reachable(const Mallocation& mallocation) const
auto value = region.read32(i * sizeof(u32));
if (value.value() == mallocation.address && !value.is_uninitialized()) {
#ifdef REACHABLE_DEBUG
dbgprintf("mallocation %p is reachable from region %p-%p\n", mallocation.address, region.base(), region.end() - 1);
report("mallocation %p is reachable from region %p-%p\n", mallocation.address, region.base(), region.end() - 1);
#endif
reachable = true;
return IterationDecision::Break;
@ -208,16 +208,16 @@ void MallocTracer::dump_leak_report()
continue;
++leaks_found;
bytes_leaked += mallocation.size;
dbgprintf("\n");
dbgprintf("==%d== \033[31;1mLeak\033[0m, %zu-byte allocation at address %#08x\n", getpid(), mallocation.size, mallocation.address);
report("\n");
report("==%d== \033[31;1mLeak\033[0m, %zu-byte allocation at address %#08x\n", getpid(), mallocation.size, mallocation.address);
Emulator::the().dump_backtrace(mallocation.malloc_backtrace);
}
dbgprintf("\n");
report("\n");
if (!leaks_found)
dbgprintf("==%d== \033[32;1mNo leaks found!\033[0m\n", getpid());
report("==%d== \033[32;1mNo leaks found!\033[0m\n", getpid());
else
dbgprintf("==%d== \033[31;1m%zu leak(s) found: %zu byte(s) leaked\033[0m\n", getpid(), leaks_found, bytes_leaked);
report("==%d== \033[31;1m%zu leak(s) found: %zu byte(s) leaked\033[0m\n", getpid(), leaks_found, bytes_leaked);
}
}