Падение Windows 10 при работе с Сом 422 портом фирмы МОХА , При закрытии приложения падает Windows 10 - появляется синий экран смерти.

[a_n_y_a]

Есть следующий код. В функции ОК падает Windows 10, появляется синий экран смерти с сообщением кода остановки системы.
Я связывался с тех поддержкой фирмы МОХА в России, они ничем помочь не могут

bool volatile Prervat422Read, IsPotok422Read;
 
Есть поток:
UINT Com422PotokRead(LPVOID pParam)
{
…
    IsPotok422Read=true;
…
while (Prervat422Read == false)
{
…  Работа с СОМ422 портом фирмы МОХА
}
...
IsPotok422Read=false;
}
 
Есть функция ОК:
 
void CPultSPNDlg::OnBnClickedOk()
{
   Prervat422Read = true;
   while (IsPotok422Read == true)  Здесь падает система.
   {
    Sleep(100);
   };
};

Что я делаю не правильно?

[Олег М]

Цитата a_n_y_a @ 22.06.17, 06:18

Что я делаю не правильно?

На мой взгляд вообще всё.
Но падает система явно не здесь. Скорее всего делаешь что-то неправильно при работе с портом, где-то в драйвере падает.

Добавлено 22.06.17, 06:37
Кстати, при нажатии OK приложение скорее всего завершается, а завершения потока ты не ждёшь. Попробуй сделать в OnBnClickedOk, после цикла, WaitForSingleObject для потока

Сообщение отредактировано: Олег М - 22.06.17, 06:37

[a_n_y_a]

Цитата Олег М @ 22.06.17, 06:32

Цитата a_n_y_a @ 22.06.17, 06:18

Что я делаю не правильно?

На мой взгляд вообще всё.
Но падает система явно не здесь. Скорее всего делаешь что-то неправильно при работе с портом, где-то в драйвере падает.

Добавлено 22.06.17, 06:37
Кстати, при нажатии OK приложение скорее всего завершается, а завершения потока ты не ждёшь. Попробуй сделать в OnBnClickedOk, после цикла, WaitForSingleObject для потока

Я не знаю как для потока сделать ожидание с помощью WaitForSingleObject.
Если можешь, напиши пожалуйста код. Буду тебе очень благодарен.

[Олег М]

Цитата a_n_y_a @ 22.06.17, 07:19

Я не знаю как для потока сделать ожидание с помощью WaitForSingleObject.
Если можешь, напиши пожалуйста код. Буду тебе очень благодарен.

HANDLE hThread = reinterpret_cast<HANDLE>(_beginthreadex.......
 
 
void CPultSPNDlg::OnBnClickedOk()
{
   Prervat422Read = true; //Сообщаем потоку, что пора завершаться
   ::WaitForSingleObject(hThread, INFINITE); //Ждём завершения потока
   ::CloseHandle(hThread); //Удаляем хендл
};

[a_n_y_a]

Цитата Олег М @ 22.06.17, 07:33

Цитата a_n_y_a @ 22.06.17, 07:19

Я не знаю как для потока сделать ожидание с помощью WaitForSingleObject.
Если можешь, напиши пожалуйста код. Буду тебе очень благодарен.

HANDLE hThread = reinterpret_cast<HANDLE>(_beginthreadex.......
 
 
void CPultSPNDlg::OnBnClickedOk()
{
   Prervat422Read = true; //Сообщаем потоку, что пора завершаться
   ::WaitForSingleObject(hThread, INFINITE); //Ждём завершения потока
   ::CloseHandle(hThread); //Удаляем хендл
};

Я сделал:

    CWinThread *Pr;
 
 
    Pr=AfxBeginThread(Com422PotokRead, this);

И далее уже

 ::WaitForSingleObject(Pr->m_..., INFINITE);

Вопрос, а удалять хендл обязательно? Если поток закрылся, может хендл удаляется автоматически?

Я поставил твой код, все работает и не падает. Большое спасибо за помощь.

[ter_nk_]

Цитата a_n_y_a @ 22.06.17, 08:18

Вопрос, а удалять хендл обязательно? Если поток закрылся, может хендл удаляется автоматически?

CloseHandle надо сделать чтобы очистить память.

[Олег М]

Цитата a_n_y_a @ 22.06.17, 08:18

Вопрос, а удалять хендл обязательно? Если поток закрылся, может хендл удаляется автоматически?

Нет, не удаляется. Будет утечка памяти, причём довольно большая - там остаётся стек потока

[a_n_y_a]

Цитата Олег М @ 22.06.17, 09:04

Цитата a_n_y_a @ 22.06.17, 08:18

Вопрос, а удалять хендл обязательно? Если поток закрылся, может хендл удаляется автоматически?

Нет, не удаляется. Будет утечка памяти, причём довольно большая - там остаётся стек потока

В моем случае достаточно сделать

  delete Pr;

Или надо еще удалять хендл:

  ::CloseHandle(Pr->m_...)

Или надо делать и то и другое?

[ter_nk_]

Цитата a_n_y_a @ 22.06.17, 06:18

while (IsPotok422Read == true) Здесь падает система.
{
Sleep(100);
};

Никогда так не делай больше!

Добавлено 22.06.17, 09:27

Цитата a_n_y_a @ 22.06.17, 09:21

Или надо еще удалять хендл:

Тебе уже говорят все. CloseHandle надо обязательно, ну и delete тоже конечно если там объект создавался динамические. Как бы если delete не сделать, то утечка памяти будет на порядки меньше.

[ЫукпШ]

Цитата Олег М @ 22.06.17, 09:04

Цитата a_n_y_a @ 22.06.17, 08:18

Вопрос, а удалять хендл обязательно? Если поток закрылся, может хендл удаляется автоматически?

Нет, не удаляется. Будет утечка памяти, причём довольно большая - там остаётся стек потока

А почему - стек ?
Хэндл можно закрыть сразу, если управлять потоком не нужно.
-----

void CPultSPNDlg::OnBnClickedOk()
{
   Prervat422Read = true; //Сообщаем потоку, что пора завершаться
   ::WaitForSingleObject(hThread, INFINITE); //Ждём завершения потока
   ::CloseHandle(hThread); //Удаляем хендл
};

Это очень неудачный вариант. Его лучше избегать.

Сообщение отредактировано: ЫукпШ - 22.06.17, 20:12

[Олег М]

Цитата ЫукпШ @ 22.06.17, 18:34

Это очень неудачный вариант. Его лучше избегать.

Почему, в чём неудачность этого варианта?

Добавлено 23.06.17, 04:42

Цитата ЫукпШ @ 22.06.17, 18:34

Хэндл можно закрыть сразу, если управлять потоком не нужно.

Вот это действительно неудачная и вредная фраза.
При вызове CloseHandle поток не будет остановлен, пока сам не завершится. Но лучше до завершения потока его не вызывать, потому что потеряешь контроль над потоком.

[a_n_y_a]

Отлаживаю программу, о которой идет речь. Программа останавливается на точке останова, я правлю код, жму Build, визуальная студия спрашивает остановить ли выполнение программы, я жму ОК, и система падает! Я работаю в VC++ 2017. Видимо те потоки, работающие с СОМ422 портом не закрываются автоматически. В этой ситуации можно что ни будь сделать?

[Олег М]

Цитата a_n_y_a @ 23.06.17, 07:57

В этой ситуации можно что ни будь сделать?

Единственно, что ты можешь здесь сделать - это выяснить падает ли система из-за какой-то твоей ошибки, или драйверы устройства так реагируют на отладчик.
Для этого её надо раз сто запустить/остановить без отладчика, если упадёт - ошибка скорее всего у тебя.
И скомпилируй в релизе, проверь иоже.

[ter_nk_]

В случае падения можно (при наличии виртуалки попробовать запустить его там), ну либо на другой машине.

[a_n_y_a]

Проверил в автономном запуске без студии. Завершаю программу через диспетчер задач. Система не падает! Дело в студии? Или диспетчер задач виртуально нажимает кнопку ОК или Cancel?