Bug 422468

Summary: Tellico crash after quit
Product: [Applications] tellico Reporter: gitdevmod <djdomics>
Component: generalAssignee: Robby Stephenson <robby>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: 3.3.1   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Microsoft Windows   
Latest Commit: Version Fixed In: 3.4
Sentry Crash Report:
Attachments: gdb backtrace

Description gitdevmod 2020-06-04 23:19:36 UTC
Created attachment 129057 [details]
gdb backtrace

SUMMARY
Tellico compiled with kde-craft.Websearch with musicbrainz and discogs and save collection works as expected.

Tellico crash when I use 'file->close' or close button.

STEPS TO REPRODUCE
1. start tellico
2. search/add stuff/save
3. quit tellico

OBSERVED RESULT
Tellico crash and maybe it could probably corrupt collection

EXPECTED RESULT
No crash

SOFTWARE/OS VERSIONS
Windows: 10 1909
KDE Frameworks 5.69.0
Qt 5.14.1 (construit sur 5.14.1)
Le système de fenêtres windows

ADDITIONAL INFORMATION
Comment 1 gitdevmod 2020-06-06 23:09:46 UTC
I've build tellico with CMAKE_DISABLE_FIND_PACKAGE_KF5NewStuff=TRUE but it still crash.
Then I commented this line "NewStuff::Manager::self();" in src/mainwindow.cpp and no more crash after quit. I do not know impact of this change. 

According to kde-craft dev, there is patch in qt for dbus on Windows and it could crash.
Is this possible to add an option to disable dbus/knewstuff in tellico ?
Comment 2 Robby Stephenson 2020-06-09 01:09:41 UTC
(In reply to gitdevmod from comment #1)
> I've build tellico with CMAKE_DISABLE_FIND_PACKAGE_KF5NewStuff=TRUE but it
> still crash.
> Then I commented this line "NewStuff::Manager::self();" in
> src/mainwindow.cpp and no more crash after quit. I do not know impact of
> this change. 

It just removes the DBUS interface for installing new themes, which you would probably not even miss. I'm not sure why it would crash though

> According to kde-craft dev, there is patch in qt for dbus on Windows and it
> could crash.
> Is this possible to add an option to disable dbus/knewstuff in tellico ?

That might be the brute-force approach. I'll work it. Thanks for working that out.
Comment 3 Robby Stephenson 2021-03-03 12:16:53 UTC
Git commit abd4e2aa357fdbf0818f875a836a0f409fc075c1 by Robby Stephenson.
Committed on 03/03/2021 at 12:16.
Pushed by rstephenson into branch 'master'.

Check dbus interface pointer before unregistered

Try to avoid crash on windows

M  +5    -1    src/newstuff/manager.cpp

https://invent.kde.org/office/tellico/commit/abd4e2aa357fdbf0818f875a836a0f409fc075c1
Comment 4 Robby Stephenson 2021-03-03 13:50:06 UTC
From my basic test, I think the recent commit fixed the crashing bug.