Skip to content

containmentitem.cpp: Do not set dropJob parent to m_dropMenu

Akseli Lahtinen requested to merge work/cherry-pick-1ffb07ed into Plasma/6.1

Setting the parent of dropJob to m_dropMenu here can cause crashes, due to m_dropMenu not necessarily existing. For example in cases where one drags an item from task manager that has no associated desktop file, the m_dropMenu is deleted, which deletes the job during async operation/running a method (line 423), which then causes crashing because things end up in odd state.

BUG:484674

(cherry picked from commit 1ffb07ed)

7beed646 containmentitem.cpp: Do not set dropJob parent to m_dropMenu 22e27f1b Remove another job->setParent(m_dropMenu)

Co-authored-by: Akseli Lahtinen akselmo@akselmo.dev

Merge request reports