Сегфолт при завершении работы потока QThread - что не так?

[xintrea]

Есть класс окна, в полях которого имеются два свойства:

Код:

 
protected:
    QThread moveDetectorThread;
    MoveDetector moveDetector; // Некий объект для запуска в потоке

В конструкторе есть следующий код, и он нормально запускает объект в потоке:

Код:

    // Запуск цикла объекта при старте потока
    connect(&moveDetectorThread, &QThread::started, &moveDetector, &MoveDetector::run);
    
    // Соединения для корректного завершения потока
    connect(&moveDetector, SIGNAL(finished()), &moveDetectorThread, SLOT(quit()));
    connect(&moveDetector, SIGNAL(finished()), &moveDetector, SLOT(deleteLater()));
    connect(&moveDetectorThread, SIGNAL(finished()), &moveDetectorThread, SLOT(deleteLater()));
    
    moveDetector.moveToThread(&moveDetectorThread); //  Объект переносится в тред
    moveDetectorThread.start(); // Тред запускается

Сам объект имеет метод основного цикла и метод, устанавливающий флаг выхода:

Код:

void MoveDetector::run()
{
    exitFlag=false;

    for(;;){
        update();
        if(exitFlag) {
            emit finished();
            return;
         }
    }
}


void MoveDetector::doExit()
{
    exitFlag=true;
}

В деструкторе окна я написал такой код:

Код:

    moveDetector.doExit();
    while(!moveDetectorThread.isFinished()) {
        qDebug() << "Wait finished move detector...";
    }

В результате, при срабатывании деструктора программа крешится вот так:

Код:

Wait finished move detector...
                 [vdso]
ffffffffff600000-ffffffffff601000 r-xp 00000000 00:00 0                  [vsyscall]
Wait finished move detector...
Wait finished move detector...
Wait finished move detector...
The program has unexpectedly finished.
Процесс был завершён принудительно.
/home/xi/work/develop/cpp/MoveNoid/build-MoveNoid-Desktop_Qt_5_9_2_GCC_64bit-Debug/MoveNoid crashed.

Информацию брал вот отсюда: https://habrahabr.ru/post/150274/

Вопрос: почему крешится программа? Что сделать чтоб поток нормально завершался?

UPD: Сделал минимальный пример: http://rgho.st/8dD7n4ljc
При нажатии Stop все зависает, окно перестает отвечать.

[Old]

Вот это уберите:

Код

C++ (Qt)
    connect(&moveDetector, SIGNAL(finished()), &moveDetector, SLOT(deleteLater()));
    connect(&moveDetectorThread, SIGNAL(finished()), &moveDetectorThread, SLOT(deleteLater()));
 

Эти объекты удалятся автоматически.

[xintrea]

Убрал, проблема осталась.

[Old]

Убрал, проблема осталась.

Давайте попробуем так:

Код

C++ (Qt)
    moveDetector.doExit();
//    while(!moveDetectorThread.isFinished()) {
//        qDebug() << "Wait finished move detector...";
//   }
    moveDetectorThread.wait();
 

[__Heaven__]

Очередность коннектов и moveToThread никак не влияет?

[Old]

Кстати, это тоже не нужно:

Код

C++ (Qt)
connect(&moveDetector, SIGNAL(finished()), &moveDetectorThread, SLOT(quit()));
 

[__Heaven__]

Кто родитель у moveDetector?

[xintrea]

Кстати, это тоже не нужно:

Код

C++ (Qt)
connect(&moveDetector, SIGNAL(finished()), &moveDetectorThread, SLOT(quit()));
 

Добавил в топик готовый минимальный пример, чтоб обсуждение было более предметно.

[xintrea]

Кто родитель у moveDetector?

QObject

[xintrea]

Давайте попробуем так:

Код

C++ (Qt)
    moveDetector.doExit();
//    while(!moveDetectorThread.isFinished()) {
//        qDebug() << "Wait finished move detector...";
//   }
    moveDetectorThread.wait();
 

Сделал:

Код:

    moveDetector.doExit();
    moveDetectorThread.wait();
    qDebug() << "Success finish.";

Проблема практически все та же - поток останавливается, но окно перестает отвечать. И сообщения о завершении нет.

[Old]

Код

C++ (Qt)
    moveDetector.doExit();
    moveDetectorThread.quit();
    moveDetectorThread.wait();
 

doExit завершает выполнения метода run MoveDetector'а, а после этого запускается цикл обработки событий самой нитки (в QThread::run вызывается метод exec()).
Вот второй quit будет завершать его.

[__Heaven__]

В примере имеется вызов слота quit.
Но до него не доходит очередь из-за цикла проверки завершения нити. processEvents доказывает это.
В таком исполнении выход происходит.

Код

C++ (Qt)
    while(!moveDetectorThread.isFinished()) {
        qApp->processEvents();
        qDebug() << "Wait finished move detector...";
    }
 

Код выше написал только для объяснения. К использованию не рекомендую.

[xintrea]

doExit завершает выполнения метода run MoveDetector'а, а после этого запускается цикл обработки событий самой нитки (в QThread::run вызывается метод exec()).
Вот второй quit будет завершать его.

Не понял. То есть, у меня из-за того, что MoveDetector::run() имеет бесконечный цикл, то обработки событий нитки вообще не происходило?

Ваше исправление:

moveDetector.doExit();
moveDetectorThread.quit();
moveDetectorThread.wait();

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

[__Heaven__]

работает, но только один раз.

Коннекты и перемещение в нить нужно в конструктор вынести

[Old]

обработки событий нитки вообще не происходило?

Да.

Еще один вариант, добавить такой коннект:

Код

C++ (Qt)
connect(&moveDetector, &MoveDetector::finished, &moveDetectorThread, &QThread::quit, Qt::DirectConnection);
 

что-бы слот QThread::quit выполнялся прямым вызовом, а не ставился в очередь.
Тогда руками quit вызывать не придется.