| Summary: | Kwave crashes when closing About dialog | ||
|---|---|---|---|
| Product: | [Applications] kwave | Reporter: | kde |
| Component: | general | Assignee: | Thomas Eschenbacher <Thomas.Eschenbacher> |
| Status: | RESOLVED WORKSFORME | ||
| Severity: | crash | ||
| Priority: | NOR | ||
| Version First Reported In: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | openSUSE | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | https://commits.kde.org/kwave/e237657b25e954060fce9419c27aebec3996c6b9 | |
| Sentry Crash Report: | |||
| Attachments: |
backtrace 1
backtrace 2 |
||
|
Description
kde
2018-05-17 08:10:23 UTC
*** Bug 399739 has been marked as a duplicate of this bug. *** Created attachment 116192 [details]
backtrace 1
Created attachment 116193 [details]
backtrace 2
I added two additional backtraces, they look a little different so I added them both. I can confirm this here, this kind of crash happens in several kinds of situations. The more interesting stuff is the text that comes out on the console: FileContext::executeCommand(): loading plugin 'about' FileContext::executeCommand(): with 1 parameter(s) ASSERT: "d->focus_next->d_func()->focus_prev == this" in file kernel/qwidget.cpp, line 1607 KCrash: Application 'kwave' crashing... It is this ASSERT that causes the trouble. It has been introduced in the KDE libraries and IMHO is useless. Does anyone know a way how to prevent this? If the assert says "kernel/qwidget.cpp", then it is an assert in Qt. OK, then it is in Qt. But I do not understand what that assert shall be good for. I have absolutely no idea what goes wrong here, and why it does not affect thousands of other applications too!? I am not aware of doing anything special there, I create a dialog, I close it, I delete the object... what should be so wrong about that? Does anyone have an idea? ok, 100% reproducible here, but only in the Help/About dialog Root cause was that a tab within a tab widget was included in the list of tab stops, as well as its parent widget. Should work now, see git commit: https://commits.kde.org/kwave/e237657b25e954060fce9419c27aebec3996c6b9 |