Skip to content

Commit 80624e0

Browse files
committed
src: fix memory leak from string conversion
missing parenthesis format c++
1 parent 6584451 commit 80624e0

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/debug_utils.cc

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -471,7 +471,7 @@ std::vector<std::string> NativeSymbolDebuggingContext::GetLoadedLibraries() {
471471
DWORD size_2 = 0;
472472
// First call to get the size of module array needed
473473
if (EnumProcessModules(process_handle, nullptr, 0, &size_1)) {
474-
MallocedBuffer<HMODULE> modules(size_1/sizeof(HMODULE);
474+
MallocedBuffer<HMODULE> modules(size_1 / sizeof(HMODULE));
475475

476476
// Second call to populate the module array
477477
if (EnumProcessModules(process_handle, modules.data, size_1, &size_2)) {
@@ -480,15 +480,15 @@ std::vector<std::string> NativeSymbolDebuggingContext::GetLoadedLibraries() {
480480
i++) {
481481
WCHAR module_name[MAX_PATH];
482482
// Obtain and report the full pathname for each module
483-
if (GetModuleFileNameW(modules.data[i],
484-
module_name,
485-
arraysize(module_name))) {
483+
if (GetModuleFileNameW(
484+
modules.data[i], module_name, arraysize(module_name))) {
486485
DWORD size = WideCharToMultiByte(
487486
CP_UTF8, 0, module_name, -1, nullptr, 0, nullptr, nullptr);
488487
char* str = new char[size];
489488
WideCharToMultiByte(
490489
CP_UTF8, 0, module_name, -1, str, size, nullptr, nullptr);
491490
list.emplace_back(str);
491+
delete str;
492492
}
493493
}
494494
}

0 commit comments

Comments
 (0)