Qt C++调用setIp后程序崩溃问题求助（UDP嗅探器场景）

问题场景 #

正在学习Qt信号与槽机制，实现了UDP嗅探器类udpSocket：

• 外部可通过setIp方法设置网卡IP
• 期望通过connect(this, &udpSocket::ipChanged, this, &udpSocket::reBind)关联信号槽，在IP变更时触发reBind重新绑定socket，维持嗅探工作

程序在初始化类后调用setIp会直接崩溃，不调用setIp则运行正常。开发环境为Qt5、MinGW C++、qmake。

崩溃原因分析 #

核心问题是类成员指针socket未正确初始化，导致野指针访问：

• 构造函数中声明了局部变量QUdpSocket *socket = new QUdpSocket;，该变量仅在构造函数作用域内有效，并未给类的成员变量socket赋值
• 成员变量socket始终处于未初始化的野指针状态
• 调用setIp时，emit ipChanged()触发reBind槽函数，reBind中执行socket->bind(...)是对野指针的非法操作，直接导致程序崩溃
• 不调用setIp时，若未触发readyRead信号（无UDP数据），readData函数不会执行，野指针问题未暴露，因此程序看似正常

修复后的代码 #

头文件（udpsocket.h） #

#include <QUdpSocket>
#include <QObject>
class udpSocket : public QObject
{
    Q_OBJECT
public slots:
    void readData();
    void reBind();
signals:
    void ipChanged();
public:
    udpSocket();
    void setIp(QString ipAddr);
private:
    QUdpSocket* socket;
    QString ip;
};

实现文件（udpsocket.cpp） #

#include "udpsocket.h"
#include <QDebug>
#include <QHostAddress>

udpSocket::udpSocket()
{
    // 直接给成员变量socket赋值，指定父对象让Qt管理内存
    socket = new QUdpSocket(this);
    // 初始化默认IP为localhost
    ip = QHostAddress(QHostAddress::LocalHost).toString();
    socket->bind(QHostAddress(ip), 443);
    
    connect(this, &udpSocket::ipChanged, this, &udpSocket::reBind);
    connect(socket, &QUdpSocket::readyRead, this, &udpSocket::readData);
}

void udpSocket::setIp(QString ipAddr){
    // 避免重复设置相同IP触发不必要的重绑定
    if (ip == ipAddr) return;
    ip = ipAddr;
    emit ipChanged();
}

void udpSocket::reBind(){
    // 先终止现有绑定，避免重复绑定冲突
    socket->abort();
    // 验证IP地址有效性，避免无效地址导致异常
    QHostAddress targetAddr(ip);
    if (targetAddr.isNull()) {
        qWarning() << "无效的IP地址：" << ip;
        // 绑定回默认地址
        targetAddr = QHostAddress::LocalHost;
        ip = targetAddr.toString();
    }
    socket->bind(targetAddr, 443);
}

void udpSocket::readData(){
    QHostAddress sender;
    quint16 senderPort;

    while(socket->hasPendingDatagrams()){
        QByteArray datagram;
        datagram.resize(socket->pendingDatagramSize());
        socket->readDatagram(datagram.data(), datagram.size(), &sender, &senderPort);
        qDebug() << datagram.data() << "IP:" << sender.toString() << "Port:" << senderPort;
    }
}

关键修复点 #

• 初始化成员指针：构造函数中直接给成员变量socket赋值，避免局部变量覆盖
• 管理socket生命周期：创建socket时指定this为父对象，利用Qt父子机制自动释放内存
• 重绑定前清理：调用socket->abort()终止现有绑定，避免重复绑定的冲突问题
• IP有效性验证：转换QHostAddress时检查是否有效，避免无效IP导致的异常
• 避免冗余操作：setIp中判断IP是否变更，避免相同IP触发不必要的重绑定

内容的提问来源于stack exchange，提问作者Andy Kostryukov