Да, этот пример не очень хорошо написан. Основной цикл службы лучше написан (концептуально) как:
// Main processing loop
while (!quit)
do_work ();
ReportSvcStatus (SERVICE_STOPPED, NO_ERROR, 0);
return;
И в обработчике обслуживания у вас есть:
// CtrlHandler callback
DWORD WINAPI CtrlHandler (DWORD dwControl, DWORD dwEventType, LPVOID lpEventData, LPVOID lpContext)
{
...
if (dwControl == SERVICE_CONTROL_STOP)
quit = true;
return NO_ERROR;
}
Где quit
глобальная переменная.
На практике (чтобы избежать цикла занятости) служба, вероятно, обычно сидит вокруг, ожидая какого-то ожидаемого объекта для чего-то делать. Представим себе, ради аргумента, это РУЧКА, называемая, скажем ghWakeupEvent
, созданная через CreateEvent()
и снова хранимая в глобальной переменной.
Затем код становится чем-то вроде:
// Main processing loop
while (1)
{
WaitForSingleObject (ghWakeupEvent, INFINITE);
if (quit)
{
ReportSvcStatus (SERVICE_STOPPED, NO_ERROR, 0);
return;
}
if (something_to_do)
do_work ();
}
// CtrlHandler callback
DWORD WINAPI CtrlHandler (DWORD dwControl, DWORD dwEventType, LPVOID lpEventData, LPVOID lpContext)
{
...
if (dwControl == SERVICE_CONTROL_STOP)
{
quit = true; // do this first!!
SetEvent (ghWakeupEvent);
}
return NO_ERROR;
}
Примечание: нет необходимости (или указывать) ghSvcStopEvent
. Пример MSDN - это путаница.