[dolphin] [Bug 504180] New: Dolphin crashes when opening a New Tab

Martín Lucas Golini <[email protected]>
Newsgroups gmane.comp.kde.devel.kfm
Message-ID <[email protected]/>
https://bugs.kde.org/show_bug.cgi?id=504180

            Bug ID: 504180
           Summary: Dolphin crashes when opening a New Tab
    Classification: Applications
           Product: dolphin
           Version: 25.04.1
          Platform: openSUSE
                OS: Linux
            Status: REPORTED
          Severity: grave
          Priority: NOR
         Component: general
          Assignee: [email protected]
          Reporter: [email protected]
                CC: [email protected]
  Target Milestone: ---

Created attachment 181261
  --> https://bugs.kde.org/attachment.cgi?id=181261&action=edit
The stack-trace

STEPS TO REPRODUCE
1. Open a New Tab with any of the methods (shortcut, menu, etc).
2. That's it.

OBSERVED RESULT
Dolphin Crashes

EXPECTED RESULT
Opened a New Tab

SOFTWARE/OS VERSIONS
Dolphin: 25.04.1 
KDE Plasma Version: 6.3.5
KDE Frameworks Version: 6.13.0
Qt Version: 6.9.0

ADDITIONAL INFORMATION
Crash I a `nullptr` exception, fix is trivial (I applied a local patch and just
checking for `m_view` not being a `nullptr` in
`DolphinViewContainer::showMessage` it's enough. Sorry for not collaborating
directly but I know nothing about KDE/Plasma development.

Tentative patch:
```
diff --git a/src/dolphinviewcontainer.cpp b/src/dolphinviewcontainer.cpp
index ee4bb6e4c..9ea049c04 100644
--- a/src/dolphinviewcontainer.cpp
+++ b/src/dolphinviewcontainer.cpp
@@ -514,7 +514,7 @@ void DolphinViewContainer::showMessage(const QString
&message, KMessageWidget::M
     m_messageWidget->animatedShow();

 #ifndef QT_NO_ACCESSIBILITY
-    if (QAccessible::isActive() && isActive()) {
+    if (QAccessible::isActive() && m_view != nullptr && isActive()) {
         // To announce the new message keyboard focus must be moved to the
message label. However, we do not have direct access to the label that is
internal
         // to the KMessageWidget. Instead we setFocus() on the KMessageWidget
and trust that it has set correct focus handling.
         m_messageWidget->setFocus();
```

-- 
You are receiving this mail because:
You are on the CC list for the bug.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.