Bug 361050

Summary:	CSV importer crashes on assigning columns (repro-file included)
Product:	[Applications] kmymoney	Reporter:	Felix Leimbach <felix.leimbach>
Component:	importer	Assignee:	KMyMoney Devel Mailing List <kmymoney-devel>
Status:	RESOLVED NOT A BUG
Severity:	normal	CC:	felix.leimbach
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Debian testing
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	CSV file for crash repro

Description Felix Leimbach 2016-03-27 17:37:57 UTC

Kmymoney version 4.7.2

Reproducible: Always

Steps to Reproduce:
1. Open the attached CSV File in the CSV Importer
2. Chose the Date-Column at set it to 1
3. kmymoney crashes

Comment 1 Felix Leimbach 2016-03-27 17:39:04 UTC

Created attachment 98115 [details]
CSV file for crash repro

Comment 2 allan 2016-03-27 18:09:50 UTC

I see only a single row, of, apparently, headings, but no actual transaction data.

Comment 3 Felix Leimbach 2016-03-27 18:17:34 UTC

That's exactly right and is enough to crash kmymoney-4.7.2 as per the "steps to reproduce" above. Obviously the real file - a paypal transaction export - contains more rows but they are not required to reproduce.

In fact we can narrow it down further: Removing all columns starting with and including "Rechnungs-Nr." in the one single line does not stop kmymoney from crashing. Remove one more column ("Vorgangs-Nr.") and it does not crash anymore.
It is not the "Vorgangs-Nr." itself though, because renaming that does not fix the crash. Also, removing some columns at the beginning can also fix the crash.

Comment 4 allan 2016-03-27 20:53:50 UTC

(In reply to Felix Leimbach from comment #3)
> That's exactly right and is enough to crash kmymoney-4.7.2 as per the "steps
> to reproduce" above. Obviously the real file - a paypal transaction export -
> contains more rows but they are not required to reproduce.
 
Not for me, I'm afraid.  No crash, no matter what column I select for the date.

Did you completely remove all the remaining rows?

Early on,  I did have problems with Paypal, because of the number of columns, but I think that should be OK now.

So, I can't go any further at the moment.

> In fact we can narrow it down further: Removing all columns starting with
> and including "Rechnungs-Nr." in the one single line does not stop kmymoney
> from crashing. Remove one more column ("Vorgangs-Nr.") and it does not crash
> anymore.
> It is not the "Vorgangs-Nr." itself though, because renaming that does not
> fix the crash. Also, removing some columns at the beginning can also fix the
> crash.

Comment 5 Felix Leimbach 2016-03-28 08:29:50 UTC

To get useful stacktraces I compiled kmymoney from kmymoney.git HEAD.
Inspecting the stacktrace revealed the problem:

The self-built kmymoney-4.7.2 (placed in ~/bin/kmymoney) loaded the kmm_csvimport.so library from /usr/lib/kde4 where Debian Testing's outdated 4.6.6 version is located!

Once I uninstalled Debian's kmymoney the correct library was loaded and the problem disappeared.

Marking RESOLVED/INVALID.