| Summary: | KMyMoney 4.8 crashes on opening some files | ||
|---|---|---|---|
| Product: | [Applications] kmymoney | Reporter: | Yuri Shirman <yshirman> |
| Component: | general | Assignee: | KMyMoney Devel Mailing List <kmymoney-devel> |
| Status: | RESOLVED DUPLICATE | ||
| Severity: | major | CC: | christian-david, mk-lists, ralf.habacker |
| Priority: | HI | Keywords: | drkonqi |
| Version: | 4.8.0 | ||
| Target Milestone: | --- | ||
| Platform: | Fedora RPMs | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/kmymoney/64d8749496a04c3be88300f040ae1c1e14af8103 | Version Fixed In: | 5.0.0 |
| Sentry Crash Report: | |||
|
Description
Yuri Shirman
2016-11-14 00:53:47 UTC
Mhh, this looks like a bug for (or caused by) me :( Can you open KMyMoney within a terminal and send me the console output? Do you use aqBanking (HBCI/FinTS)? If not, you can disable the aqbanking plugin in the settings. This work around could solve the crash. (In reply to Christian David from comment #1) Yes, disabling kbanking plugin solves the issue for me. Thanks. Here is the console output: KMyMoney csvexport plugin loaded onlineTask available "org.kmymoney.creditTransfer.germany" onlineTaskConverter available "org.kmymoney.creditTransfer.germany" ("org.kmymoney.creditTransfer.sepa") onlineTaskConverter available "org.kmymoney.creditTransfer.sepa" ("org.kmymoney.creditTransfer.germany") KMyMoney ofximport plugin loaded KMyMoney kbanking plugin loaded KMyMoney reconciliation report plugin loaded reading file start parsing file startDocument reading accounts reading transactions reading securities reading currencies reading prices reading reports endDocument kmymoney: buffer.c:1017: GWEN_Buffer_AppendString: Assertion `buffer' failed. KCrash: Application 'kmymoney' crashing... KCrash: Attempting to start /usr/libexec/kde4/drkonqi from kdeinit KCrash: Connect sock_file=/home/shirman/.kde/socket-woland.lan/kdeinit4__0 According to https://scan.coverity.com/projects/kmymoney?tab=overview has kmymoney 4.8 about 80 issues related to null pointer deferences. It may help to fix them. Git commit 64d8749496a04c3be88300f040ae1c1e14af8103 by Christian Dávid. Committed on 15/11/2016 at 18:09. Pushed by christiand into branch 'master'. Fixed constructors in KBanking Some class members were not initialized correctly what potentially caused crashes. FIXED-IN: 5.0 M +11 -2 kmymoney/plugins/kbanking/mymoneybanking.cpp http://commits.kde.org/kmymoney/64d8749496a04c3be88300f040ae1c1e14af8103 (In reply to Ralf Habacker from comment #3) > According to https://scan.coverity.com/projects/kmymoney?tab=overview has > kmymoney 4.8 about 80 issues related to null pointer deferences. It may help > to fix them. That is probably right. I made a commit for KMyMoney 5 to fix this. It should be easy to back port the patch (unfortunately I cannot make/test it at the moment). Then we need to evaluate if the bug was actually fixed or if this is an upstream bug. However, this shows again that the plugins should have their own thread. A crash of a plugin (even if we give it invalid data) should not take down KMyMoney. (In reply to Christian David from comment #5) > (In reply to Ralf Habacker from comment #3) > > According to https://scan.coverity.com/projects/kmymoney?tab=overview has > > kmymoney 4.8 about 80 issues related to null pointer deferences. It may help > > to fix them. > > That is probably right. I made a commit for KMyMoney 5 to fix this. It > should be easy to back port the patch I back ported it to 4.8 with minor fixes (do not use nullptr). > (unfortunately I cannot make/test it at the moment). Then we need to evaluate if the bug was actually fixed or if this is an upstream bug. How can we check that ? KDE's CI system takes care of those checks. (Or, did I misunderstand you?) (In reply to Marko Käning from comment #7) > KDE's CI system takes care of those checks. (Or, did I misunderstand you?) This bug has been reported by an end user with a dedicated configuration. I guess checking the related functionality is not covered by a unit test running on the CI. (In reply to Ralf Habacker from comment #8) > I guess checking the related functionality is not covered by a unit > test running on the CI. Sorry for the noise, then. :) (In reply to Christian David from comment #4) > Git commit 64d8749496a04c3be88300f040ae1c1e14af8103 by Christian Dávid. > Committed on 15/11/2016 at 18:09. > Pushed by christiand into branch 'master'. > > Fixed constructors in KBanking > > Some class members were not initialized correctly what potentially > caused crashes. > FIXED-IN: 5.0 > > M +11 -2 kmymoney/plugins/kbanking/mymoneybanking.cpp > > http://commits.kde.org/kmymoney/64d8749496a04c3be88300f040ae1c1e14af8103 I suffer from the same bug. However the proposed patch did not solve the issue. Neither did disabling the banking plugin. How can I help to solve the bug. What information would you need therefore? Corrigendum: disabling the kbanking plugin works, just needed to restart kmymoney. fixed in bug #378435 - you may close this - works for me now *** This bug has been marked as a duplicate of bug 378435 *** |