smw/deskflow
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(daemon): Validate process handle and ID before shutdown

Browse source
This commit is contained in:
Nick Bolton 2025-04-15 23:10:46 +01:00 committed by Chris Rizzitello
parent 901dacf593
commit 8a2b6bd688
1 changed files with 10 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/lib/platform/MSWindowsProcess.cpp
View file

@ -121,6 +121,16 @@ void MSWindowsProcess::shutdown(HANDLE handle, DWORD pid, int timeout)
{ {
LOG_DEBUG("shutting down process %d", pid); LOG_DEBUG("shutting down process %d", pid);
if (handle == nullptr) {
LOG_ERR("invalid process handle, cannot shutdown process %d", pid);
return;
}
if (pid == 0) {
LOG_ERR("invalid process id, cannot shutdown process %d", pid);
return;
}
LOG_DEBUG("sending close event to close process gracefully"); LOG_DEBUG("sending close event to close process gracefully");
HANDLE hCloseEvent = OpenEvent(EVENT_MODIFY_STATE, FALSE, kCloseEventName); HANDLE hCloseEvent = OpenEvent(EVENT_MODIFY_STATE, FALSE, kCloseEventName);
if (hCloseEvent != nullptr) { // NOSONAR -- Readability if (hCloseEvent != nullptr) { // NOSONAR -- Readability