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

Nicolas Fella <nicolas.fe...@gmx.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/fram
                   |                            |eworks/knotifications/commi
                   |                            |t/d1bb15dc4892d26201d21f123
                   |                            |62979736df1d50c
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #12 from Nicolas Fella <nicolas.fe...@gmx.de> ---
Git commit d1bb15dc4892d26201d21f12362979736df1d50c by Nicolas Fella.
Committed on 12/01/2021 at 12:33.
Pushed by nicolasfella into branch 'master'.

Fix removing pending notification from queue

If a notification is in the pending queue it is not yet in the
notifications map. When the notification is then closed we will hit the early
return because the id wasn't found.

We need to still remove it from the queue so move the relevant block up.

If the notification is not removed from the queue it will stay as a
dangling pointer and blow up when the capabilities are queried and we
reemit the notification.

M  +8    -8    src/notifybypopup.cpp

https://invent.kde.org/frameworks/knotifications/commit/d1bb15dc4892d26201d21f12362979736df1d50c

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

Reply via email to