复制到 QMap 时出现 QString 错误。范围问题？

Question

每当我尝试将 QString 添加到 QMap 作为 QWidget 类的析构函数中的键时，我目前都会遇到段错误（有时由于错误的 malloc 导致的 sigabrts） - 我认为这与 QString 的隐式共享模型有关范围。

我有一个 QWidget 充​​当 MDI 中的子窗口，该 QWidget 有一些 QGLWidget 派生视口实例作为子项。在子窗口中有一个QMap< QString、QVariant >包含项目文件设置的包装类，当子窗口关闭时，它的析构函数调用 QWidget::deleteChildren() 来删除每个视口。在视口析构函数中，当前设置保存到子窗口的设置中，例如：

QString dir = QString( "camera/" ) + name;
sWin.setSetting( dir + "/Projection",
                 static_cast< int >( camera_->getProjectionType() ) );

为我希望保存的每个属性调用 sWin.setSetting()，'sWin' 是对 QMap 包装类的引用，该类在子窗口析构函数结束。一切都很好，直到 setSetting() 调用，这只是：

inline void setSetting( const QString& key, QVariant value )
{
    //  projectSettings_ is a standard QMap< QString, QVariant >.
    projectSettings_.insert( key, value );
}

此设置对于第一个视口工作正常，在第二个视口的第一次 setSetting() 调用时，出现段错误：

inline QString::QString(const QString &other) : d(other.d)
{ Q_ASSERT(&other != this); d->ref.ref(); }

我是否看到当我的 QString 引用传递给 QMap 时，深层复制失败？如果是这样，为什么？我创建的 QString 尚未超出范围，因为析构函数尚未返回。为什么这会在第二个视口而不是第一个视口上失败？

有时，我会在第一个代码示例的 setSetting() 行中得到一个 sigabrt ma​​lloc():内存损坏 in operator+ 。但同样，在第二个视口被破坏开始时，而不是第一个视口被破坏时。

对于这个非常冗长的问题，我深表歉意，但是涉及到的大量代码分布在许多翻译单元中。这个问题的任何线索都会有很大的帮助！

提前致谢。 Cam

---

更新

我将第一个代码示例更改为：

QString* dir = new QString( "camera/" );
*dir += name;
sWin.setSetting( *dir + "/Projection",
                 static_cast< int >( camera_->getProjectionType() ) );

作为范围问题的测试。它有时有效，而其他时候则给出完全相同的错误；所以推测堆 QString 正在被破坏，这仅取决于它的内存位置是否已被覆盖。但这也没有意义，因为我没有对其调用删除，也没有关闭应用程序（只是一个 MDI 子窗口）！

---

更多代码

当实例化新的子窗口时，sWin 会在堆上创建。它被带入视口类的析构函数中，作为我的子窗口中的方法的引用：

inline Sy_project& getProject() { return *project_; }

Sy_project 是（当前）QMap 的简单包装类。我完整的视口基类析构函数是这样的（它在这里失败，而不是在派生类中）：

Sy_abstractGLViewport::~Sy_abstractGLViewport()
{
    //  Write last viewport settings.
    QString name = objectName();
    Sy_project& sWin = projWindow_->getProject();
    QString dir = QString( "camera/" ) + name;

    //  Avoid "taking address of temporary" warning.
    QVector3D pos = camera_->getPosition();
    QVector3D foc = camera_->getFocalPoint();

    //  Save camera settings.  Dies on first setSetting() call.
    sWin.setSetting( dir + "/Projection",
                     static_cast< int >( camera_->getProjectionType() ) );
    sWin.setSetting( dir + "/position",
                     QVariant( 84, static_cast< void* >( &pos ) ) );
    sWin.setSetting( dir + "/focalPoint",
                     QVariant( 84, static_cast< void* >( &foc ) ) );
    sWin.setSetting( dir + "/FOV", camera_->getFOV() );
    sWin.setSetting( dir + "/nearClip", camera_->getPerspectiveClipRange()[0] );
    sWin.setSetting( dir + "/farClip", camera_->getPerspectiveClipRange()[1] );

    delete camera_;
}

---

Valgrind

在使用 Valgrind 的 memcheck 之后，我发现了许多看起来与我的堆栈跟踪类似的条目。以前从未使用过它，我仍在破译，但这是否是说我的 Sy_project 类（QMap 的包装器）在 setSetting() 调用后已被删除，从而使 QMap 具有无效引用？

==12418== Invalid read of size 4
==12418==    at 0x805D872: QMap<QString, QVariant>::detach_helper() (qmap.h:730)
==12418==    by 0x805D380: QMap<QString, QVariant>::detach() (in /home/cbamber85/workspace/Syren GUI/Syren)
==12418==    by 0x805CDEE: QMap<QString, QVariant>::insert(QString const&, QVariant const&) (qmap.h:537)
==12418==    by 0x805CA33: Sy_project::setSetting(QString const&, QVariant) (Sy_project.h:50)
==12418==    by 0x805A78C: Sy_abstractGLViewport::~Sy_abstractGLViewport() (Sy_abstractGLViewport.cpp:67)
==12418==    by 0x808EDBC: Sy_QtGLViewport::~Sy_QtGLViewport() (Sy_QtGLViewport.cpp:91)
==12418==    by 0x808EE0E: Sy_QtGLViewport::~Sy_QtGLViewport() (Sy_QtGLViewport.cpp:100)
==12418==    by 0x4D66D63: QObjectPrivate::deleteChildren() (in /usr/lib/libQtCore.so.4.6.3)
==12418==    by 0x4306DDF: QWidget::~QWidget() (in /usr/lib/libQtGui.so.4.6.3)
==12418==    by 0x46FBE0E: QFrame::~QFrame() (in /usr/lib/libQtGui.so.4.6.3)
==12418==    by 0x475F173: QSplitter::~QSplitter() (in /usr/lib/libQtGui.so.4.6.3)
==12418==    by 0x475F1D1: QSplitter::~QSplitter() (in /usr/lib/libQtGui.so.4.6.3)
==12418==  Address 0xacf5b9c is 4 bytes inside a block of size 8 free'd
==12418==    at 0x40266AD: operator delete(void*) (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==12418==    by 0x808D60F: Sy_project::~Sy_project() (Sy_project.h:30)
==12418==    by 0x808C9C6: Sy_subWindow::~Sy_subWindow() (Sy_subWindow.cpp:55)
==12418==    by 0x808CA84: Sy_subWindow::~Sy_subWindow() (Sy_subWindow.cpp:57)
==12418==    by 0x4D66482: qDeleteInEventHandler(QObject*) (in /usr/lib/libQtCore.so.4.6.3)
==12418==    by 0x4D67967: QObject::event(QEvent*) (in /usr/lib/libQtCore.so.4.6.3)
==12418==    by 0x4302ACB: QWidget::event(QEvent*) (in /usr/lib/libQtGui.so.4.6.3)
==12418==    by 0x42A9C63: QApplicationPrivate::notify_helper(QObject*, QEvent*) (in /usr/lib/libQtGui.so.4.6.3)
==12418==    by 0x42B1CA3: QApplication::notify(QObject*, QEvent*) (in /usr/lib/libQtGui.so.4.6.3)
==12418==    by 0x806010F: Sy_application::notify(QObject*, QEvent*) (Sy_application.cpp:14)
==12418==    by 0x4D54E0D: QCoreApplication::notifyInternal(QObject*, QEvent*) (in /usr/lib/libQtCore.so.4.6.3)
==12418==    by 0x4D589B3: QCoreApplicationPrivate::sendPostedEvents(QObject*, int, QThreadData*) (in /usr/lib/libQtCore.so.4.6.3)

Answer 1

我认为您的指针和引用所指向的 Sy_project 对象在调用 Sy_abstractGLViewport 的析构函数时已经被销毁。如果您查看 valgrind 列表，会发现 Sy_project 的析构函数在 Sy_abstractGLViewport 的析构函数之前调用。

因此，当您调用内联 Sy_project& getProject() { 返回 *project_; } 在 Sy_abstractGLViewport 析构函数中，您正在取消引用悬空指针。

Answer 2

嗯...如果您将代码更改为以下内容，会发生什么：

dir.append("/Projection");
sWin.setSetting(dir, static_cast<int>(camera_->getProjectionType()));

按照您执行临时副本的方式，创建一个临时副本，然后绑定到对 const 的引用，这应该没问题。之后，QMap 将尝试从该引用复制参数，在本例中，将通过 QString 的隐式共享机制。我只是想知道里面会出什么问题...