QTcpSocket SIGSEGV при попытке реконекта после краша сервера

@BITniki · Регистрация: 07.02.2022

Учу QT. Пытаюсь реализовать удалённое управление мышкой через телефон с помощью QTcpServer и QTcpSocket. Однако при реконекте выпадает SIGSEGV

Вот конструктор клиента:

C++ (Qt)

MainWindow::MainWindow(QWidget *parent)
    : QMainWindow(parent)
    , ui(new Ui::MainWindow)
{
    ui->setupUi(this);
    setAttribute(Qt::WA_AcceptTouchEvents, true);
 
    socket = new QTcpSocket(this);
 
    connect(socket, &QTcpSocket::connected, this, &MainWindow::connectResived);
    connect(socket, &QTcpSocket::disconnected, socket, &QTcpSocket::deleteLater);
    connect(socket, &QTcpSocket::disconnected, this, &MainWindow::disconnectResived);
}

По нажатии кнопки pushButton, вызывается этот слот:

C++ (Qt)

void MainWindow::on_pushButton_clicked()
{
    socket->connectToHost("192.168.31.193", 2323);
 
}

Последовательность действий такова:
Запускается сервер и клиент
жмём pushButton и подключаемся
Закрываем сервер
Запускаем сервер заново/не запускаем и жмём pushButton
SIGSEGV

В процессе гугления я наткнулся на это решение, однако то ли знаний в инглише не хватает, то ли знаний в qt, но я его не понял

Собственно я пытался сделать что либо при получении сигнала disconnected() но это ничего не изменило:

C++ (Qt)

void MainWindow::disconnectResived()
{
    socket->disconnectFromHost();
    //ну или
    QMetaObject::invokeMethod(m_socket, "connectToHost", Qt::QueuedConnection);
}

@Ender Che · 09.02.2022, 00:27

А вот это пробовали (с той же ссылки стэковерфлоу):

For the ones who may come across with a similar situation, the reset method of the QTcpSocket solved the problem.

т.е. вот это:

C++ (Qt)

1	[virtual] bool QIODevice::reset()

?

@Annemesski · 09.02.2022, 07:38

у Вас в конструкторе клиента у сокета связан сигнал disconnected со слотом deleteLater, а значит при переходе сокета в состояние "отключен" он уничтожается и далее в своем слоте on_bushbutton_clicked Вы (при повторном подключении) разыменовываете уаазатель на разрушенный объект и получаете закономерный сегментэйшн фолт. Перенесите строки с созданием сокета и подключением сигналов из конструктора в слот on_button_clicked и вставьте их перед вызовом connectToHost.

@kapbepucm · 09.02.2022, 09:31

А если после разрыва соединения и при удалении сокета создавать опять новый QTcpSocket?

@Annemesski · 09.02.2022, 12:30

kapbepucm, похоже на подход проктолога в зубоврачебном кабинете, зачем создавать новый сокет при разрыве соединения? А если в дальнейшем соединение вовсе не будет устанавливаться вплоть до окончания работы программы, тогда вообще зачем содержать в памяти сконструированный сокет? Логичнее сконструировать сокет непосредственно перед установкой соединения. Хотя в данном конкретном случае вообще непонятно зачем создавать сокет в куче, уже куда лучше было бы объявить сокет непосредственным членом класса и отдать управление памятью на откуп непосредственно RAII и вместо связывания сигналов по deleteLater написать слот-обработчик для сигнала void QTcpSocket::stateChanged(QAbstractSocket::SocketState socketState), тем более что в текущем варианте, даже если перенести конструирование сокета и его связку с сигналами в слот-обработчик кнопки, то это хоть и уберет проблему с падение программы, но потенциально грозит утечкой памяти, так как если в ходе подключения к серверу произойдет какая-нибудь ошибка сокета, то сокет встанет в ошибку и выбросит stateChange, а disconnected так и не будет выброшен и сокет не будет удален, а при повторном нажатии на кнопку создастся новый сокет и перетрет старый указатель.

@kapbepucm · 09.02.2022, 13:35

Не по теме:

Annemesski, я невнимательно прочитал задание, почемуто решил, что речь идёт об автоматическом reconnect при обрыве связи :gsorry:

@BITniki · 09.02.2022, 16:51 **[ТС]**

kapbepucm, спасибо, ваш совет помог.

Сообщение от Annemesski

kapbepucm, Хотя в данном конкретном случае вообще непонятно зачем создавать сокет в куче, уже куда лучше было бы объявить сокет непосредственным членом класса и отдать управление памятью на откуп непосредственно RAII и вместо связывания сигналов по deleteLater написать слот-обработчик для сигнала void QTcpSocket::stateChanged(QAbstractSocket::SocketState socketState)

А можно поподробнее про это? Создание в куче это socket = new QTcpSocket(this);? Что значит объявить членом класса?

@Annemesski · 09.02.2022, 16:56

Сообщение от BITniki

Создание в куче это socket = new QTcpSocket(this);?

Да, это оно. Память выделяемая вручную, выделяется в куче (грубо говоря в оперативной памяти).

Сообщение от BITniki

Что значит объявить членом класса?

Объявить членом класса объект типа QTcpSocket, а не указатель на него, тогда объект будет создан вместе с классом и уничтожен вместе с ним.

@Annemesski 2436 / 1176 / 435 Регистрация: 08.11.2016 Сообщений: 3,249
	09.02.2022, 07:38	3
	Сообщение было отмечено BITniki как решение Решение у Вас в конструкторе клиента у сокета связан сигнал disconnected со слотом deleteLater, а значит при переходе сокета в состояние "отключен" он уничтожается и далее в своем слоте on_bushbutton_clicked Вы (при повторном подключении) разыменовываете уаазатель на разрушенный объект и получаете закономерный сегментэйшн фолт. Перенесите строки с созданием сокета и подключением сигналов из конструктора в слот on_button_clicked и вставьте их перед вызовом connectToHost. 2

@kapbepucm 1322 / 585 / 283 Регистрация: 02.05.2020 Сообщений: 1,409
	09.02.2022, 09:31	4
	А если после разрыва соединения и при удалении сокета создавать опять новый QTcpSocket? 0

@Annemesski 2436 / 1176 / 435 Регистрация: 08.11.2016 Сообщений: 3,249
	09.02.2022, 12:30	5
	kapbepucm, похоже на подход проктолога в зубоврачебном кабинете, зачем создавать новый сокет при разрыве соединения? А если в дальнейшем соединение вовсе не будет устанавливаться вплоть до окончания работы программы, тогда вообще зачем содержать в памяти сконструированный сокет? Логичнее сконструировать сокет непосредственно перед установкой соединения. Хотя в данном конкретном случае вообще непонятно зачем создавать сокет в куче, уже куда лучше было бы объявить сокет непосредственным членом класса и отдать управление памятью на откуп непосредственно RAII и вместо связывания сигналов по deleteLater написать слот-обработчик для сигнала `void QTcpSocket::stateChanged(QAbstractSocket::SocketState socketState)`, тем более что в текущем варианте, даже если перенести конструирование сокета и его связку с сигналами в слот-обработчик кнопки, то это хоть и уберет проблему с падение программы, но потенциально грозит утечкой памяти, так как если в ходе подключения к серверу произойдет какая-нибудь ошибка сокета, то сокет встанет в ошибку и выбросит stateChange, а disconnected так и не будет выброшен и сокет не будет удален, а при повторном нажатии на кнопку создастся новый сокет и перетрет старый указатель. 3

@kapbepucm
	09.02.2022, 13:35 #6
	Не по теме: Annemesski, я невнимательно прочитал задание, почемуто решил, что речь идёт об автоматическом reconnect при обрыве связи :gsorry: 0

@BITniki 0 / 0 / 0 Регистрация: 07.02.2022 Сообщений: 15
	09.02.2022, 16:51 [ТС]	7
	kapbepucm, спасибо, ваш совет помог. Сообщение от Annemesski kapbepucm, Хотя в данном конкретном случае вообще непонятно зачем создавать сокет в куче, уже куда лучше было бы объявить сокет непосредственным членом класса и отдать управление памятью на откуп непосредственно RAII и вместо связывания сигналов по deleteLater написать слот-обработчик для сигнала `void QTcpSocket::stateChanged(QAbstractSocket::SocketState socketState)` А можно поподробнее про это? Создание в куче это `socket = new QTcpSocket(this);`? Что значит объявить членом класса? 0

@Annemesski 2436 / 1176 / 435 Регистрация: 08.11.2016 Сообщений: 3,249
	09.02.2022, 16:56	8
	Сообщение от BITniki Создание в куче это socket = new QTcpSocket(this);? Да, это оно. Память выделяемая вручную, выделяется в куче (грубо говоря в оперативной памяти). Сообщение от BITniki Что значит объявить членом класса? Объявить членом класса объект типа QTcpSocket, а не указатель на него, тогда объект будет создан вместе с классом и уничтожен вместе с ним. 2

Опции темы

QTcpSocket SIGSEGV при попытке реконекта после краша сервера

Решение