| Summary: | Crash in CSV importer due to too much columns in CSV file (e.g. paypal CVS files) | ||
|---|---|---|---|
| Product: | [Applications] kmymoney | Reporter: | Ian Hubbertz <kde_bugs> |
| Component: | general | Assignee: | KMyMoney Devel Mailing List <kmymoney-devel> |
| Status: | RESOLVED FIXED | ||
| Severity: | crash | CC: | agander93, kde_bugs, onet.cristian |
| Priority: | NOR | ||
| Version: | 4.6.3 | ||
| Target Milestone: | --- | ||
| Platform: | Ubuntu | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
| Attachments: | New crash information added by DrKonqi | ||
|
Description
Ian Hubbertz
2013-02-11 22:13:42 UTC
Created attachment 77173 [details]
New crash information added by DrKonqi
kmymoney (4.6.3) on KDE Platform 4.9.3 using Qt 4.8.3
- What I was doing when the application crashed:
Same crash, now with setting the amount column to value '8' (seems to be col=7 in backtrace) instead of date column to value '1' (col=0 in BT) as in the previous backtrace.
-- Backtrace (Reduced):
#5 0x00007f7f4f887936 in QString::operator== (this=0x46047a0, other=...) at tools/qstring.cpp:2192
#6 0x00007f7f3824208d in CsvImporterDlg::validateColumn (this=this@entry=0x4604370, col=@0x7fff94121f1c: 7, type=...) at /build/buildd/kmymoney-4.6.3/kmymoney/plugins/csvimport/csvimporterdlg.cpp:265
#7 0x00007f7f38242860 in CsvImporterDlg::amountColumnSelected (this=0x4604370, col=7) at /build/buildd/kmymoney-4.6.3/kmymoney/plugins/csvimport/csvimporterdlg.cpp:288
[...]
#9 0x00007f7f4ee425f1 in QComboBox::currentIndexChanged (this=this@entry=0x46a4eb0, _t1=7) at .moc/release-shared/moc_qcombobox.cpp:316
#10 0x00007f7f4ee42657 in QComboBoxPrivate::_q_emitCurrentIndexChanged (this=this@entry=0x46a4f00, index=...) at widgets/qcombobox.cpp:1278
Haha.. found it. csvprocessing.h:41 #define MAXCOL 25 // maximum no. of columns (arbitrary value) Seems that this value is not large enough for paypal CSV files... (In reply to comment #2) > Haha.. found it. > > csvprocessing.h:41 > > #define MAXCOL 25 // maximum no. of columns (arbitrary > value) > > > Seems that this value is not large enough for paypal CSV files... OK. Well spotted. Snag is, that file is not part of kmymoney (4.6.3). So, the question is, where did it come from? It might be that it's a relic of 4.6.2, which was not uninstalled, and the old plugin is still in your system. In what style does the plugin appear? Is it the tabs version or the wizard? Have you compiled from source, or was it a deb you had previously? In 4.6.3 the file is still in use, see https://projects.kde.org/projects/extragear/office/kmymoney/repository/show/kmymoney/plugins/csvimport?rev=4.6.3 Does the new importer in the master branch allow more columns? BTW: As workaround for the problem it is possible to select the columns at the paypal site before CSV export and reduce the number of colums to less than 20. (In reply to comment #4) > In 4.6.3 the file is still in use, see Apologies, a slip of the tongue. The problem was fixed last August, in the development release. That too involved a Paypal file. It's likely that 4.6.4 will soon be released. > https://projects.kde.org/projects/extragear/office/kmymoney/repository/show/ > kmymoney/plugins/csvimport?rev=4.6.3 > Does the new importer in the master branch allow more columns? Yes, there is no longer a hard-coded limit. > > BTW: As workaround for the problem it is possible to select the columns at > the paypal site before CSV export and reduce the number of colums to less > than 20. That's handy. Fixed in master as stated by Allan. |