[webkit-reviews] review requested: [Bug 32614] [Qt] Memory leak detected in ThreadingQt : [Attachment 45563] fix crash in patch 2

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Dec 28 10:54:09 PST 2009


Chang Shu <Chang.Shu at nokia.com> has asked  for review:
Bug 32614: [Qt] Memory leak detected in ThreadingQt
https://bugs.webkit.org/show_bug.cgi?id=32614

Attachment 45563: fix crash in patch 2
https://bugs.webkit.org/attachment.cgi?id=45563&action=review

------- Additional Comments from Chang Shu <Chang.Shu at nokia.com>
calling deleteLater in detachThread may not guarantee the thread is finished
while being deleted. Using a slot function to capture the finished() signal and
calling deleteLater from there is safe.


More information about the webkit-reviews mailing list