QTcpSocket::readAll() SIGSEGV

[RedDog]

В TCP сервере сокет пускается в отдельном потоке:

Код:

void CSocketThread::run()
{
    client = QSharedPointer< QTcpSocket >( new QTcpSocket() );
    client->setSocketDescriptor( socketDescriptor );
    QObject::connect( client.data(), SIGNAL( readyRead() ),
                      this, SLOT( on_dataReceived() ), Qt::QueuedConnection );
    exec();
}

Код:

void CSocketThread::on_dataReceived()
{
    QTcpSocket* socket = qobject_cast< QTcpSocket* >( sender() );
    if( !socket )
    {
        return;
    }

    int bytesToRead = socket->bytesAvailable();

    QByteArray tmp = socket->read( bytesToRead );
......
......
......
}

Вот в самом сервере создается поток для сокета.

Код:

void CTcpServer::incomingConnection( qintptr handle )
{
    client = QSharedPointer< CSocketThread >( new CSocketThread( handle ) );
    QObject::connect( client.data(), &CSocketThread::dataReceived,
                      this, &CTcpServer::on_dataReceived );
    QObject::connect( client.data(), &QThread::finished,
                      client.data(), &QObject::deleteLater );
    client->start();
}

При чтении из сокета возникает SIGSEGV. Вылетает в случайном порядке. По объему данных никакой корреляции не увидел. Крайний раз bytesToRead = 608
ЧЯДНТ?
PS: Qt 5.3.0

[kuzulis]

ЧЯДНТ?

Переменная QTcpSocket живет в потоке CSocketThread::run(), но bytesAvailable() и read() выполняются из главного.

ЗЫ: Ну сколько же раз будут еще такие темы подниматься? Самому проверить и подумать не судьба?

[RedDog]

Переменная QTcpSocket живет в потоке CSocketThread::run(), но bytesAvailable() и read() выполняются из главного.

Тогда встречный вопрос: как сделать что бы они в том же потоке выполнялись?

[torwig]

В потоке, в котором создаете сокет, делаете все connect() и слоты там же пусть отрабатывают. Абсолютно всю работу с сокетом выносите в тот поток, где он был создан.

[RedDog]

В потоке, в котором создаете сокет, делаете все connect() и слоты там же пусть отрабатывают. Абсолютно всю работу с сокетом выносите в тот поток, где он был создан.

Так у меня так все и сделано. В run() создаю сокет, там же делаю коннекты к слотам.

[qate]

ЧЯДНТ?

наследуешся от qthread ?

https://wiki.qt.io/Threads_Events_QObjects

[Bepec]

qate поясните пожалуйста.

Возможно человек просто не знал, что при наследовании от QThread необходимо в конструктор добавлять moveToThread(this), дабы слоты и остальные ф-ции принадлежали потоку, а не оставались в потоке родителя.

[qate]

qate поясните пожалуйста.

я имел ввиду, что методика не наследоваться от qthread описано в http://doc.qt.io/qt-5/qthread.html

чем плох moveToThread(this) не могу сказать

[Авварон]

чем плох moveToThread(this) не могу сказать

Код:

It is important to remember that a QThread instance lives in the old thread that instantiated it, not in the new thread that calls run(). 
This means that all of QThread's queued slots will execute in the old thread. 
Thus, a developer who wishes to invoke slots in the new thread must use the worker-object approach; 
new slots should not be implemented directly into a subclassed QThread.

[Авварон]

чем плох moveToThread(this) не могу сказать

Код:

It is important to remember that a QThread instance lives in the old thread that instantiated it, not in the new thread that calls run(). 
This means that all of QThread's queued slots will execute in the old thread. 
Thus, a developer who wishes to invoke slots in the new thread must use the worker-object approach; 
new slots should not be implemented directly into a subclassed QThread.

Типа после moveToThread(this) у вас контрроллер треда (QThread) будет жить в треде, к-ый он контролирует и, по-хорошему, все его ф-ии (quit/wait) надо будет звать из этого потока. Типа если будете дергать из главного потока, то работать будет, но некошерно.

[Bepec]

Столько копий было сломано, но в результате вывод - это равнозначные методы применения

[RedDog]

И все равно не понимаю...
Объект создал в потоке, подконнектил его там же.
Так почему слоты вызываются в другом потоке то?

[Igors]

И все равно не понимаю...
Объект создал в потоке, подконнектил его там же.
Так почему слоты вызываются в другом потоке то?

Потому что так работает Qt::AutoConnection

[RedDog]

Потому что так работает Qt::AutoConnection

Однако, у меня Qt::QueuedConnection

[kuzulis]

Qt::QueuedConnection

Все верно в этом случае - слот будет вызван из главного потока приложения. Можно попробовать задать DirectConnection (я не уверен что это правильный путь), но
тогда придется "защищать" прочитанные данные tmp, если планируется их передавать в основной поток:

Код

C++ (Qt)
...
int bytesToRead = socket->bytesAvailable();
QByteArray tmp = socket->read( bytesToRead );
...
 
// здесь надо что-то делать с tmp!