[kdiagram] [Bug 383160] Crash on showing investment price chart

[Dag Andersen]

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

--- Comment #7 from Dag Andersen <danders at get2net.dk> ---
(In reply to Ralf Habacker from comment #6)
> (In reply to Ralf Habacker from comment #5)
> > -           position.row >=0 && position.row < m_data[ 0 ].size();
> > -> this checks if the requested row is in column 0, which may be wrong if 
> On the crash I had 
> 
> m_data[0].size was 367
> m_data[1].size was 110
> position.column was 1
> position.row was 110
If I understand item models correctly, this should be impossible as all rows
with the same parent must have the same column count, so you should get the
same number of rows for all columns.
> 
> The original codes checkes that position.row < m_data[ 0 ].size() which
> returns true (where it should return false) but fails later on access to
> m_data[1][110] with an out of index exception.
There are many places in the code it is assumed that all column have the same
tow count.

I'm afraid this will just hide a problem eslewhere.
I have tested master branch with the attached file but cannot trigger a crash.
More info will be appricated.

-- 
You are receiving this mail because:
You are on the CC list for the bug.