Крешь на TerminateProcess

[PSD]

Есть приложение своеобразный шетдулер который мониторить приборы и по некоторому "сложному" алгоритму стратует задачи.
В том случае если по какйто причине запущеная им задача зависла и не успела завершится к моенту следующего запуска он должен убить ее и запустить заново  для этого вставлен следующий кусок
  

Код:

DAC_Stat d=m_Counts[id];
TRY
{
	this->WriteLog (hFileLog,CString(_T("Terminate old DL proc ")) );
	TerminateProcess(sRun.hProc ,0);
	this->WriteLog (hFileLog,CString(_T("Terminate sucsses ")) );
}CATCH_ALL(pE)
{   
	this->WriteLog (hFileLog,CString(_T("Terminate error ")) );
}END_CATCH_ALL; 

Приложение  в некий произволный момент оработав произвольное число циклов вылетает, в логе последней строчкой всегда

"Terminate old DL proc "


В чем проблема?

[Антон (LogRus)]

ты уверен, что вот это указывает на сторонний процесс, а не на текущий
первое, что в голову пришло

[PSD]

Отладчиком не смотрел ибо до времени креша прога может работать 3-4 часа, дальше там:

 if( !CreateProcess( NULL, // No module name (use command line).
        b, // Command line.
        NULL,             // Process handle not inheritable.
        NULL,             // Thread handle not inheritable.
        FALSE,            // Set handle inheritance to FALSE.
        0,                // No creation flags.
        NULL,             // Use parent's environment block.
        NULL,             // Use parent's starting directory.
        &si,              // Pointer to STARTUPINFO structure.
        &pi )             // Pointer to PROCESS_INFORMATION structure.
    )
    {
      printf("\n DAC: DL hour not run. [%d-%d-%d %d:%d:%d],[%d] ", d.LTLD.GetYear(),  d.LTLD.GetMonth(),d.LTLD.GetDay(),d.LTLD.GetHour(),d.LTLD.GetMinute(),d.LTLD.GetSecond(),d.id_count_DB);
       sRun.hProc=INVALID_HANDLE_VALUE;
      return 0;
    }
 sRun.hProc =pi.hProcess;


Еще одно время у меня система на одну одна из задач запускаемых этим шетдулером, выдавала сообщение о  дедлоке....   

[Антон (LogRus)]

а ты убей внешний процесс, что бы условие убийства сработало.
поробуй значение
sRun.hProc
кидать в лог, при установке и перед уничтожением.
вдруг они не совпадают
попробуй ибивать сразу после старта.

[PSD]

Записать хендел в лог поробую, после старта убивается без проблем , и на отладчике убивается а когад находитсяв работе после нескольких часов работы вылетает и именновэтом месте....

[Diletant]

К моменту удаления Handle уже не соответствует процессу и существует только потому, что ты его не закрыл. Имеет смысл хранить не handle, а PiD и получать хэндл заново с помощью OpenProcess().

[PSD]

К моменту удаления Handle уже не соответствует процессу и существует только потому, что ты его не закрыл. Имеет смысл хранить не handle, а PiD и получать хэндл заново с помощью OpenProcess().

Я тоже об этом думал, но предпологал что  TerminateProcess просто обязан коректно обрабатывать такую ситуацию, в противном случае его былобы в принципе нельзя использовать, потому что как не крути ноо всегда есть вероятность что между  OpenProcess() и TerminateProcess процесс завершится сам....