https://bugs.kde.org/show_bug.cgi?id=458933

Stefano Crocco <stefano.cro...@alice.it> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED
      Latest Commit|                            |https://invent.kde.org/netw
                   |                            |ork/konqueror/commit/c9f3d1
                   |                            |957f7b6c168a025b5b4014e46a5
                   |                            |f49bb63

--- Comment #6 from Stefano Crocco <stefano.cro...@alice.it> ---
Git commit c9f3d1957f7b6c168a025b5b4014e46a5f49bb63 by Stefano Crocco.
Committed on 21/09/2022 at 17:27.
Pushed by dfaure into branch 'master'.

Fix crash caused by using original view rather than actual view

UrlLoader::decideEmbedOrSave used the original view when deciding
whether it could be reused to open the URL, instead of using the view
provided using UrlLoader::setView. If the URL was opened by clicking on
a sidebar item, this caused a sidebar view to be used, rather than the
correct dolphin part. This lead to a crash because the sidebar view is
a passive view, which caused KonqViewManager::chooseNextView to return
nullptr

M  +1    -1    src/konqmainwindow.cpp
M  +6    -8    src/urlloader.cpp

https://invent.kde.org/network/konqueror/commit/c9f3d1957f7b6c168a025b5b4014e46a5f49bb63

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to