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

Michael Pyne <mp...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://commits.kde.org/juk
                   |                            |/e8c2dbbfce00765335d079bca8
                   |                            |6abc7e2147f3e6
             Status|CONFIRMED                   |RESOLVED
   Version Fixed In|                            |17.12.2
         Resolution|---                         |FIXED

--- Comment #2 from Michael Pyne <mp...@kde.org> ---
Git commit e8c2dbbfce00765335d079bca86abc7e2147f3e6 by Michael Pyne.
Committed on 19/01/2018 at 03:18.
Pushed by mpyne into branch 'Applications/17.12'.

Immediately update dynamic/search playlist items upon a change.

This is as opposed to recording the new list of PlaylistItems and then
effecting the change after draining the event loop (an attempt at
keeping GUI responsive if many updates are happening).

Otherwise it is possible to
- Generate a list of PlaylistItems to read from (when updating the
playlist) and push to end of event loop
- Delete some of those PlaylistItems due to already-queued events to
delete playlist items
- Finally try to use the list of PlaylistItems and run into a segfault
from trying to access a deleted PlaylistItem.

I think this should fix a crash when you delete multiple playlists.
FIXED-IN:17.12.2

M  +3    -5    dynamicplaylist.cpp

https://commits.kde.org/juk/e8c2dbbfce00765335d079bca86abc7e2147f3e6

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

Reply via email to