Skip to content

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

BUG: 458933

Merge request reports