Quote:
Originally Posted by sendersu
I"ve ran the PVS Tool against the tool,
I"d like to note -very well done!
some minor suggestions: https://prnt.sc/vQpQVeHUiyV0
eg:
DWORD GetProcessIdByName(wstring processName)
->
DWORD GetProcessIdByName(const wstring& processName)
and similar ones
2) there are some potential set of handle leaks (not closed)
3) I've commented out this enablement:
//EnablePrivilege(SE_DEBUG_NAME);
and the tool still works?!
4) imagine that some end user is having localized Windows OS, eg chinese or japanese or some arabic, etc
how about supporting these users as well?
IMHO in this case the routine
std::string GetLastErrorAsString()
need to be unicode (wide string) aware
|
please post upload your compilation cause first one does not work well on my pc with win 11 pro 23h2