search

Azure / vld

Visual Leak Detector for Visual C++ 2008-2015
https://kinddragon.github.io/vld/
GNU Lesser General Public License v2.1
42 stars 25 forks source link

Fix for unreported memory leaks with Win32 applications #16

Open avadae opened 2 years ago

avadae commented 2 years ago

I've reverted a change in utility.cpp that caused memory leaks not to be reported in x86 builds.

A simple leak like this: 

int main()
{
    int* x = new int {10};
}

was not reported, now it is again. In the meantime I also updated to vs2022

ghost commented 2 years ago

CLA assistant check
All CLA requirements met.

Neustradamus commented 3 months ago

@avadae, @mattdurak: Any progress on it?

mattdurak commented 3 months ago 
//}

VirtualQuery was too slow, QueryVirtualMemoryInformation addressed performance issues.

We are not supporting x86 so any fix here cannot degrade the perf for x64.

Refers to: src/utility.cpp:1298 in 3e46ec8. [](commit_id = 3e46ec8c4ab47acd0fe633d8f20b4682c17a4209, deletion_comment = False)

grrava commented 1 week ago

This merge request can be closed - I'll run my own version. It is indeed affecting performance but I do need the x86 version to work.