Bug 361050 - CSV importer crashes on assigning columns (repro-file included)
Summary: CSV importer crashes on assigning columns (repro-file included)
Status: RESOLVED NOT A BUG
Alias: None
Product: kmymoney
Classification: Applications
Component: importer (show other bugs)
Version: unspecified
Platform: Debian testing Linux
: NOR normal
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-03-27 17:37 UTC by Felix Leimbach
Modified: 2016-03-28 08:29 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
CSV file for crash repro (646 bytes, text/csv)
2016-03-27 17:39 UTC, Felix Leimbach
Details

Note You need to log in before you can comment on or make changes to this bug.
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.