Summary: | KMyMoney 5.2.0 crash when updating online stock and currency prices | ||
---|---|---|---|
Product: | [Applications] kmymoney | Reporter: | supernobby <andreas_barth> |
Component: | general | Assignee: | KMyMoney Devel Mailing List <kmymoney-devel> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | ralf.habacker |
Priority: | NOR | ||
Version First Reported In: | 5.2.0 | ||
Target Milestone: | --- | ||
Platform: | Arch Linux | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | 5.2.1 | |
Sentry Crash Report: | |||
Attachments: | test file |
Description
supernobby
2025-06-30 12:39:05 UTC
(In reply to supernobby from comment #0) ... > Thread 1 (Thread 0x7fffd23eb5c0 (LWP 5933) "kmymoney"): > #0 AlkOnlineQuotesProfile::hasGHNSSupport (this=0x0) at > /usr/src/debug/libalkimia/alkimia-8.2.0/src/alkonlinequotesprofile.cpp:180 > No locals. > #1 0x00007ffff739ba50 in AlkOnlineQuoteSource::read (this=0x7fffffffc748) > at /usr/src/debug/libalkimia/alkimia-8.2.0/src/alkonlinequotesource.cpp:359 > No locals. A look at the corresponding source code lines shows that there is still potential for improvement here to check whether a profile is available. > #2 0x00007ffff7f74fc4 in KEquityPriceUpdateDlgPrivate::addInvestment > (this=this@entry=0x55555855a8d0, inv=...) at > /usr/src/debug/kmymoney/kmymoney-5.2.0/kmymoney/dialogs/ > kequitypriceupdatedlg.cpp:371 but the real problem lies here, because here a null pointer is passed for the profile, while an available profile is required. AlkOnlineQuoteSource alkOnlineSource(inv.value(QLatin1String("kmm-online-source")), quoteProfile(inv.value(QLatin1String("kmm-online-quote-system")))); Git commit 6a793773cb8471db5a921d1951302f8b0ee5957c by Ralf Habacker. Committed on 30/06/2025 at 20:20. Pushed by habacker into branch 'master'. Guard access to m_profile member of class AlkOnlineQuoteSource::Private M +11 -0 autotests/alkonlinequotesourcetest.cpp M +1 -0 autotests/alkonlinequotesourcetest.h M +7 -6 src/alkonlinequotesource.cpp M +10 -0 src/alkonlinequotesource_p.h https://invent.kde.org/office/alkimia/-/commit/6a793773cb8471db5a921d1951302f8b0ee5957c Created attachment 182826 [details]
test file
Does it work with the attached test file and this recipe?
1. open kmymoney
2. load the attached file
3. go to the settings page for online quotes
4. install the quote source for “Yahoo.com [currency]”.
5. select the menu item “Update stock and currency rates”.
6. Press "Update all"
Git commit 234c8eae9d9960c2994b49cff9154810a09fb9d7 by Thomas Baumgart, on behalf of Ralf Habacker. Committed on 09/07/2025 at 11:41. Pushed by tbaumgart into branch '8.2'. Guard access to m_profile member of class AlkOnlineQuoteSource::Private (cherry picked from commit 6a793773cb8471db5a921d1951302f8b0ee5957c) M +11 -0 autotests/alkonlinequotesourcetest.cpp M +1 -0 autotests/alkonlinequotesourcetest.h M +7 -6 src/alkonlinequotesource.cpp M +10 -0 src/alkonlinequotesource_p.h https://invent.kde.org/office/alkimia/-/commit/234c8eae9d9960c2994b49cff9154810a09fb9d7 (In reply to Ralf Habacker from comment #3) > Created attachment 182826 [details] > test file > > Does it work with the attached test file and this recipe? > > 1. open kmymoney > 2. load the attached file > 3. go to the settings page for online quotes > 4. install the quote source for “Yahoo.com [currency]”. > 5. select the menu item “Update stock and currency rates”. > 6. Press "Update all" Sorry for the late reply. I tried this and it worked. |