[kmymoney4] [Bug 384021] Crash on investment charts with multiple investments

[Ralf Habacker]

https://bugs.kde.org/show_bug.cgi?id=384021

Ralf Habacker <ralf.habacker at freenet.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
      Latest Commit|                            |https://commits.kde.org/kmy
                   |                            |money/a583dc9d434ec1f42bc41
                   |                            |7769adcf3449dea741c
   Version Fixed In|                            |4.8.1
         Resolution|---                         |FIXED

--- Comment #3 from Ralf Habacker <ralf.habacker at freenet.de> ---
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

-- 
You are receiving this mail because:
You are the assignee for the bug.