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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/netw |https://invent.kde.org/netw
                   |ork/konqueror/commit/429936 |ork/konqueror/commit/2d0501
                   |da48ebd8bcc6f75dacd9180eb08 |01105638f5d72f1e3486d80a0ea
                   |6e4add3                     |d1fd721

--- Comment #9 from Stefano Crocco <stefano.cro...@alice.it> ---
Git commit 2d050101105638f5d72f1e3486d80a0ead1fd721 by Stefano Crocco.
Committed on 07/10/2022 at 17:20.
Pushed by stefanocrocco into branch 'release/22.08'.

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/2d050101105638f5d72f1e3486d80a0ead1fd721

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

Reply via email to