Closing multiple notifications crashes unity8
Bug #1453958 reported by
Gustavo Pichorim Boiko
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical System Image |
Fix Released
|
High
|
kevin gunn | ||
| unity-notifications |
Fix Released
|
High
|
Lukáš Tinkl | ||
Bug Description
If a notification client places multiple notifications and ask them to be removed all at once, unity8 crashes.
The attached python3 script reproduces the problem.
You might need to install python3-notify2 for the script to work.
Related branches
lp:~lukas-kde/unity-notifications/fix-1453958
- Pete Woods: Approve
-
Diff: 422 lines (+145/-93)9 files modifiedinclude/ActionModel.h (+4/-4)
include/Notification.h (+3/-0)
include/NotificationModel.h (+3/-3)
include/NotificationPlugin.h (+2/-2)
src/Notification.cpp (+3/-3)
src/NotificationModel.cpp (+45/-37)
src/NotificationServer.cpp (+6/-6)
test/CMakeLists.txt (+0/-1)
test/notificationtest.cpp (+79/-37)
Revision history for this message
| Gustavo Pichorim Boiko (boiko) wrote | #1 |
| summary: |
- Closing multiple notifications crash unity8 + Closing multiple notifications crashes unity8 |
Revision history for this message
| Tiago Salem Herrmann (tiagosh) wrote | #2 |
Revision history for this message
| Gustavo Pichorim Boiko (boiko) wrote | #3 |
Revision history for this message
| Mirco Müller (macslow) wrote | #4 |
| Changed in unity-notifications: | |
| status: | New → Confirmed |
| importance: | Undecided → High |
| assignee: | nobody → Mirco Müller (macslow) |
| Changed in unity-notifications: | |
| status: | Confirmed → In Progress |
| Changed in canonical-devices-system-image: | |
| milestone: | none → ww46-2015 |
| assignee: | nobody → Kevin Gunn (kgunn) |
| assignee: | Kevin Gunn (kgunn) → nobody |
| assignee: | nobody → kevin gunn (kgunn72) |
| importance: | Undecided → High |
| Changed in unity-notifications: | |
| assignee: | Mirco Müller (macslow) → nobody |
| Changed in canonical-devices-system-image: | |
| status: | New → In Progress |
| Changed in unity-notifications: | |
| assignee: | nobody → Lukáš Tinkl (lukas-kde) |
| Changed in unity-notifications: | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | In Progress → Fix Committed |
| Changed in canonical-devices-system-image: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
It is worth noting that the crash is not reproducible if you remove the notifications in the same order they were created, this is why the clear() method is using reversed(range(20)) instead of range(20).