101696 2005-03-17 15:27:44 +0000 korganizer crash at startup for a particular std.ics file 2006-12-16 11:21:52 +0000 1 1 2 Applications korganizer general unspecified Debian testing Linux RESOLVED FIXED NOR crash --- 1 bertrand.haut pim-bugs-null 0 oldest_to_newest 324779 0 bertrand.haut 2005-03-17 15:27:44 +0000 Version: (using KDE KDE 3.3.2) Installed from: Debian testing/unstable Packages OS: Linux If I try to launch korganizer, I get the following error : berti@venus:~/code/kynaptic-kde/src$ korganizer ERROR: Communication problem with korganizer, it probably crashed. After some research, it appears that if I remove my ~/.kde/share/apps/korganizer/std.ics file, korganizer is working. I've succeed to reduc my std.ics file to two entries. If these entries are there, korganizer crash. If only one of the two is removed (any of the two), korganizer is working. In attachment, I put the std.ics file. 324781 1 10163 bertrand.haut 2005-03-17 15:29:26 +0000 Created attachment 10163 the bogus std.ics file 370299 2 me 2005-09-01 23:43:52 +0000 Still crashes in today's 3.5 branch. 482550 3 reinhold 2006-11-02 18:51:06 +0000 Reassigning all KOrganizer bug reports and wishes to the newly created korganizer-devel mailing list. 494063 4 martin 2006-12-16 11:21:52 +0000 SVN commit 614095 by mkoller: BUG: 101696 Break recursive relation to avoid endless loop M +14 -1 kotodoview.cpp --- branches/KDE/3.5/kdepim/korganizer/kotodoview.cpp #614094:614095 @@ -590,6 +590,9 @@ // Use dynamic_cast, because in the future the related item might also be an event Todo *relatedTodo = dynamic_cast<Todo *>(incidence); + // just make sure we know we have this item already to avoid endless recursion (Bug 101696) + mTodoMap.insert(todo,0); + // kdDebug(5850) << " has Related" << endl; QMap<Todo *,KOTodoViewItem *>::ConstIterator itemIterator; itemIterator = mTodoMap.find(relatedTodo); @@ -599,7 +602,17 @@ } // isn't this pretty stupid? We give one Todo to the KOTodoViewItem // and one into the map. Sure finding is more easy but why? -zecke - KOTodoViewItem *todoItem = new KOTodoViewItem(*itemIterator,todo,this); + KOTodoViewItem *todoItem; + + // in case we found a related parent, which has no KOTodoViewItem yet, this must + // be the case where 2 items refer to each other, therefore simply create item as root item + if ( *itemIterator == 0 ) { + todo->setRelatedTo(0); // break the recursion, else we will have troubles later + todoItem = new KOTodoViewItem(mTodoListView,todo,this); + } + else + todoItem = new KOTodoViewItem(*itemIterator,todo,this); + return mTodoMap.insert(todo,todoItem); } else { // kdDebug(5850) << " no Related" << endl; 10163 2005-03-17 15:29:26 +0000 2005-03-17 15:29:26 +0000 the bogus std.ics file std.ics.bug application/octet-stream 723 bertrand.haut QkVHSU46VkNBTEVOREFSClBST0RJRDotLy9LIERlc2t0b3AgRW52aXJvbm1lbnQvL05PTlNHTUwg bGlia2NhbCAzLjIvL0VOClZFUlNJT046Mi4wCgoKCkJFR0lOOlZUT0RPCkRUU1RBTVA6MjAwNTAz MTBUMTcwOTIwWgpPUkdBTklaRVI7Q049QmVydHJhbmQgSGF1dDpNQUlMVE86aGF1dEBhdXRvLnVj bC5hYy5iZQpDUkVBVEVEOjIwMDUwMzAyVDE0MjgzMVoKVUlEOmxpYmtjYWwtMTQxODQ2MTkxNC42 NzIKU0VRVUVOQ0U6MgpMQVNULU1PRElGSUVEOjIwMDUwMzEwVDE2MzgwN1oKU1VNTUFSWTpjYXJ0 ZSBkJ2lkZW50aXRpw6kKQ0xBU1M6UFVCTElDClBSSU9SSVRZOjMKUkVMQVRFRC1UTzpsaWJrY2Fs LTYwMzczMjk4LjcyMgpQRVJDRU5ULUNPTVBMRVRFOjUwCkVORDpWVE9ETwoKQkVHSU46VlRPRE8K RFRTVEFNUDoyMDA1MDMxMFQxNzA5MjBaCk9SR0FOSVpFUjtDTj1CZXJ0cmFuZCBIYXV0Ok1BSUxU TzpoYXV0QGF1dG8udWNsLmFjLmJlCkNSRUFURUQ6MjAwNTAzMDJUMTQyODU5WgpVSUQ6bGlia2Nh bC02MDM3MzI5OC43MjIKU0VRVUVOQ0U6MApMQVNULU1PRElGSUVEOjIwMDUwMzEwVDE2MzgxMloK U1VNTUFSWTphc3N1cmFuY2UgZXQgY2hhbmdlbWVudCBkJ2FkZHJlc3NlcyBJTkcKQ0xBU1M6UFVC TElDClBSSU9SSVRZOjMKUkVMQVRFRC1UTzpsaWJrY2FsLTE0MTg0NjE5MTQuNjcyClBFUkNFTlQt Q09NUExFVEU6MzAKRU5EOlZUT0RPCgoKRU5EOlZDQUxFTkRBUgoK