Summary: | crash when starting kontact after login [QSharedDataPointer<Akonadi::EntityPrivate>::operator=, Akonadi::Entity::operator=, operator=] | ||
---|---|---|---|
Product: | [Frameworks and Libraries] Akonadi | Reporter: | Marc Schiffbauer <mschiff> |
Component: | libakonadi | Assignee: | Volker Krause <vkrause> |
Status: | RESOLVED WORKSFORME | ||
Severity: | crash | ||
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
Marc Schiffbauer
2009-12-22 03:15:17 UTC
Volker, could it be a symptom of the bug I fixed with r1060671? That one also had an invalid memory access at QSharedDataPointer<Akonadi::EntityPrivate>::operator=. That fix should also be merged to trunk. SVN commit 1066751 by tmcguire: SVN_MERGE CCBUG: 219626 Merged revisions 1060404,1060670-1060671 via svnmerge from svn+ssh://tmcguire@svn.kde.org/home/kde/branches/work/akonadi-ports/kdepimlibs ........ r1060404 | tmcguire | 2009-12-08 21:04:13 +0100 (Tue, 08 Dec 2009) | 3 lines Add test for walking the parent collections after parsing a collection. Fails because of some uninitalized memory reads. ........ r1060670 | tmcguire | 2009-12-09 16:35:06 +0100 (Wed, 09 Dec 2009) | 5 lines Cleanup the test. I hope adjusting the HRID test was ok, I guess adding another test caused the negative collections to be even more negative. ........ r1060671 | tmcguire | 2009-12-09 16:37:59 +0100 (Wed, 09 Dec 2009) | 8 lines Fix crash in ProtocolHelperTest::testParentCollectionAfterCollectionParsing(). See comments for details. This is the best fix I could come up with, I hope it doesn't hurt performance too much. (And the lesson we have learned: Don't cast pointers to references and then return them) Volker, you'll want to review this. ........ _M . (directory) M +25 -3 akonadi/entity.cpp M +6 -0 akonadi/entity.h M +20 -2 akonadi/tests/protocolhelpertest.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1066751 The merge should have fixed the bug, please reopen if you can still reproduce it. |