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: dolphin-bugs-n...@kde.org
          Reporter: spart...@gmail.com
                CC: kfm-de...@kde.org
  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 watching all bug changes.

Reply via email to