|
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
Last edited by sendersu; 10-12-2024 at 17:42.
|