| Summary: | Crash on investment charts with multiple investments | ||
|---|---|---|---|
| Product: | [Applications] kdiagram | Reporter: | David Houlden <djhoulden> |
| Component: | KChart | Assignee: | Dag Andersen <dag.andersen> |
| Status: | RESOLVED FIXED | ||
| Severity: | crash | CC: | ralf.habacker |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | https://commits.kde.org/kmymoney/a583dc9d434ec1f42bc417769adcf3449dea741c | Version Fixed In: | 4.8.1 |
| Attachments: |
Backtrace
Anon file to demonstrate problem |
||
You have a test case or a how to reproduce ? Created attachment 107532 [details]
Anon file to demonstrate problem
Open the anon file attached. In investment reports, R000013, 14, 16 will crash KMyMoney. R000018 will work.
Git commit a583dc9d434ec1f42bc417769adcf3449dea741c by Ralf Habacker. Committed on 31/08/2017 at 17:47. Pushed by habacker into branch '4.8'. Fix 'Crash on investment charts with multiple investments' The root cause of this crash is an implementation bug in CartesianDiagramDataCompressor::slotColumnsAboutToBeInserted(). In case m_model is set, adding columns to m_data requires to add a QVector<DataPoint> instance sized to the model row count. Instead the size is limited to m_xResolution which depends on the diagram zoom factor and other values. Having a too small sized column in m_data let slotRowsAboutToBeInserted() crash. This patch needs to be pushed upstream to have it in further kdiagram releases. Fixup of commit 7516308b. FIXED-IN:4.8.1 M +1 -1 libkdchart/src/KDChart/Cartesian/KDChartCartesianDiagramDataCompressor_p.cpp https://commits.kde.org/kmymoney/a583dc9d434ec1f42bc417769adcf3449dea741c Fix has been added to kmymoneys 4.8 kdiagram source copy for now. Afaics this fix is included in KChart 2.6 so closing it. |
Created attachment 107521 [details] Backtrace Latest 4.8 git head crashes whenever I try to display an investment price graph which shows price history for more than one investment. Charts showing only one investment work ok. Backtrace attached.