Summary: | Soprano lockups (freezes) in trunk, e.g. with Konqueror | ||
---|---|---|---|
Product: | nepomuk | Reporter: | Christoph Feck <cfeck> |
Component: | general | Assignee: | Sebastian Trueg <sebastian> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | ronisbr |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Unspecified | ||
Latest Commit: | Version Fixed In: |
Description
Christoph Feck
2010-02-10 23:42:32 UTC
I can confirm that using Soprano r1082162 fixes the lockups again (with the rest of the system current trunk). SVN commit 1088551 by trueg: unlock the mutex before deleting it. BUG: 226218 M +2 -1 clientconnection.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1088551 *** Bug 226154 has been marked as a duplicate of this bug. *** commit bebb50871961ec19431ceb250e349b42be8e4aea branch 2.5 Author: Sebastian Trueg <sebastian@trueg.de> Date: Thu Feb 11 09:25:44 2010 +0000 unlock the mutex before deleting it. BUG: 226218 svn path=/trunk/kdesupport/soprano/; revision=1088551 diff --git a/client/clientconnection.cpp b/client/clientconnection.cpp index e75e70b..2d2fed8 100644 --- a/client/clientconnection.cpp +++ b/client/clientconnection.cpp @@ -71,12 +71,13 @@ Soprano::Client::ClientConnection::ClientConnection( QObject* parent ) Soprano::Client::ClientConnection::~ClientConnection() { - QMutexLocker lock( &d->socketMutex ); + d->socketMutex.lock(); // the sockets need to be deleted in their respective threads. // this is what d->socketStorage does. We only close them here. foreach( QIODevice* socket, d->sockets ) { socket->close(); } + d->socketMutex.unlock(); delete d; } |