]> cloud.milkyroute.net Git - dolphin.git/commitdiff
Delete FileMetaDataToolTip using deleteLater() to prevent crashes
authorFrank Reininghaus <frank78ac@googlemail.com>
Sat, 24 Nov 2012 04:08:25 +0000 (05:08 +0100)
committerFrank Reininghaus <frank78ac@googlemail.com>
Sat, 24 Nov 2012 04:10:56 +0000 (05:10 +0100)
The problem was that Qt tried to deliver an 'Enter' event to the
dangling pointer that once pointed to the tool tip.

Many thanks to G. Christ for finding a way to reproduce this crash!

BUG: 282257
BUG: 310579
FIXED-IN: 4.9.4

src/views/tooltips/tooltipmanager.cpp

index d948bfd866cef405e432ed2a67b85b4a85b28a5c..bd6948363e58c94ce68afacc4ffb94d25b5b809e 100644 (file)
@@ -104,7 +104,9 @@ void ToolTipManager::hideToolTip()
 
     if (m_fileMetaDataToolTip) {
         m_fileMetaDataToolTip->hide();
-        delete m_fileMetaDataToolTip;
+        // Do not delete the tool tip immediately to prevent crashes when
+        // QCoreApplication tries to deliver an 'Enter' event to it, see bug 310579.
+        m_fileMetaDataToolTip->deleteLater();
         m_fileMetaDataToolTip = 0;
     }
 }